Skip to content

docs(configuration): document HINDSIGHT_API_RETAIN_CHUNK_BATCH_SIZE#1045

Merged
nicoloboschi merged 1 commit intovectorize-io:mainfrom
mrkhachaturov:chore/docs-retain-chunk-batch-size
Apr 14, 2026
Merged

docs(configuration): document HINDSIGHT_API_RETAIN_CHUNK_BATCH_SIZE#1045
nicoloboschi merged 1 commit intovectorize-io:mainfrom
mrkhachaturov:chore/docs-retain-chunk-batch-size

Conversation

@mrkhachaturov
Copy link
Copy Markdown
Contributor

Summary

retain_chunk_batch_size and its HINDSIGHT_API_RETAIN_CHUNK_BATCH_SIZE env var have lived on HindsightConfig since the retain streaming batch landed, but the Retain section of the configuration reference never got a row for them. Anyone who wants to cap chunk-batch size on large document ingestion has to grep the source to find the env var name.

Adds one row to the Retain table, in the same format as the surrounding entries, with a note that the field is configurable per bank via the bank config API.

Test plan

  • Row format matches the surrounding HINDSIGHT_API_RETAIN_* entries (variable | description | default).
  • CI green — docs site build validates the markdown table renders.

The retain_chunk_batch_size hierarchical config field and its
ENV_RETAIN_CHUNK_BATCH_SIZE loader have existed in HindsightConfig
since the retain streaming batch landed, but the Retain section of
the configuration reference never got a row for them — users who
want to cap chunk-batch size on large document ingestion had to
discover the env var by grepping the source.

Add a row to the Retain table next to the other chunk/batch knobs,
with the same format as surrounding entries and an explicit note
that the field is configurable per bank via the bank config API.
@mrkhachaturov mrkhachaturov force-pushed the chore/docs-retain-chunk-batch-size branch from 9e411aa to ba6b500 Compare April 14, 2026 07:10
Copy link
Copy Markdown
Collaborator

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@nicoloboschi nicoloboschi merged commit cf99188 into vectorize-io:main Apr 14, 2026
86 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants