Repository navigation
docs(spicepod): Flight batch_size and ipc_compression, OTLP temporality, runtime.tracing, and the tokenizerConfig file type - #2443
Conversation
…mporality, runtime.tracing, and the tokenizerConfig model file type
✅ Pull with Spice PassedPassing checks:
|
|
🚀 deployed to https://4fdaa511.spiceai-org-website.pages.dev |
There was a problem hiding this comment.
🟡 Changes recommended
The low_memory description misstates counter temporality in all three runtime references, which could lead readers to interpret exported metrics incorrectly.
1 open issue
What changed in this PR
This documentation-only PR fills gaps in the Spicepod configuration reference and corrects the tokenizer configuration file type to match the runtime’s accepted value.
Changes:
- Documents Flight batch sizing and IPC compression, OTLP temporality, and Zipkin tracing for vNext, 2.4.x, and 2.3.x.
- Corrects
tokenizer_configtotokenizerConfigin all 13 model references.
| File | Description |
|---|---|
website/versioned_docs/version-2.4.x/reference/spicepod/runtime.md |
Documents missing runtime settings for 2.4.x. |
website/versioned_docs/version-2.4.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-2.3.x/reference/spicepod/runtime.md |
Documents missing runtime settings for 2.3.x. |
website/versioned_docs/version-2.3.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-2.2.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-2.1.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-2.0.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-1.9.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-1.8.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-1.7.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-1.6.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-1.5.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-1.11.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/versioned_docs/version-1.10.x/reference/spicepod/models.md |
Corrects the tokenizer file type. |
website/docs/reference/spicepod/runtime.md |
Documents missing runtime settings for vNext. |
website/docs/reference/spicepod/models.md |
Corrects the tokenizer file type. |
Review effort: Balanced
…ers and histograms
|
Copilot review #2443 (review) ( |
|
🚀 deployed to https://2f20c13f.spiceai-org-website.pages.dev |

Summary
A Gaps sweep of the
spicepod-schemainventory compared.schema/spicepod.schema.jsonand thecrates/spicepod/srcstructs onspiceai/spiceaiorigin/trunkwithwebsite/docs/reference/spicepod/. This PR fills five of the gaps it found. Four are Spicepod properties with no row on any docs page. The fifth is an enum value that the reference spells in a form the runtime does not accept.Each property exists unchanged at
v2.4.0andv2.3.3, so the rows go into the three maintained versions: vNext, 2.4.x, and 2.3.x. Thefiles[].typevalue is wrong in every version, so that correction goes into all 13models.mdcopies.runtime.telemetry.otel_exporter.temporality:delta(default),cumulative, orlow_memory.runtime.flight.batch_size:mode: adaptive(default, withmax: 131072) ormode: default.runtime.flight.ipc_compression:none(default),lz4_frame, orzstd.runtime.tracing:zipkin_enabledandzipkin_endpoint. Before this PR it was documented only on the Zipkin monitoring page.models[].files[].type: the reference listedtokenizer_config.ModelFileTypeis#[serde(rename_all = "camelCase")]with no alias, so the accepted value istokenizerConfig.Reproduction
Item 5:
tokenizer_configvstokenizerConfig(Reproduced, name claim)Doc said (
website/docs/reference/spicepod/models.md:103at docs31f71082, and the same line in everyversioned_docs/version-*/reference/spicepod/models.md):- `tokenizer_config`: Tokenizer configurationCode says (
crates/spicepod/src/component/model.rsatorigin/trunk814e639e):Same at every release the docs cover:
Behavior claim, that a Spicepod with
type: tokenizer_configfails to load: not run, because there is nospicedbuild on this machine. The claim follows from serde'srename_all = "camelCase"with no alias. Unverified, code inspection only.Items 1–4: missing rows (Gaps, additions)
The sweep found no row for these properties on any page:
Verification
Read on
origin/trunk814e639e8eb55eacbe3f3fadaad2ec0aaf712c05. Each struct field is also present atv2.4.0andv2.3.3:Files changed: 16 (3
runtime.mdand 13models.md).Propagation:
Build:
NODE_OPTIONS=--max-old-space-size=12288 npm run build:Style grep over the added lines: no hits.
Test plan
cd website && npm run buildpasses (output above)Review gate
codexnot installed (receipt:attempt=codex result=not-installed,engine=none)/security-review: not run — docs-only Markdown diff; the same review on the sibling docs-only PR docs: request-context metric labels, HTTPS redirect guard, ORC metadata limit, scalar function memory, OTLP log record size #2440 found nothing/simplify: not run — docs-only diff, no code