Skip to content

fix: Document missing pg_replication_bootstrap_batch_size parameter#1744

Merged
lukekim merged 1 commit into
trunkfrom
fix/postgres-replication-bootstrap-batch-size
May 19, 2026
Merged

fix: Document missing pg_replication_bootstrap_batch_size parameter#1744
lukekim merged 1 commit into
trunkfrom
fix/postgres-replication-bootstrap-batch-size

Conversation

@claudespice
Copy link
Copy Markdown
Collaborator

Summary

The PostgreSQL data connector accepts pg_replication_bootstrap_batch_size to control rows-per-batch during the initial CDC snapshot, but it was missing from both the connector reference and the CDC feature page.

Changes

Add pg_replication_bootstrap_batch_size (default 8192, max 1048576) to:

  • website/docs/components/data-connectors/postgres/index.md — replication parameters table
  • website/docs/features/cdc/postgres-replication.md — full configuration reference

Reference

Verified against spiceai/spiceai trunk:

Version scope

The parameter was added in source on 2026-05-12 (commit f4a5089df, "Improve CDC ingestion performance"). It is not yet contained in any release tag, so only the unversioned vNext docs need updating — versioned doc sets (1.5.x through 1.11.x) intentionally untouched.

Test plan

  • Verified parameter name, default, and maximum against the ParameterSpec declaration and optional_usize_in_range consumer call site.
  • Confirmed the parameter is end-to-end wired (declared in PARAMETERS, consumed in replication.rs, used in bootstrap.rs) — not a non-functional name.
  • Local Docusaurus build skipped: host disk had ~300Mi free, insufficient for the ~600MB build output. The change is two new rows in existing tables, no new links or frontmatter tags, so broken-link / undefined-tag checks are not affected.

The PostgreSQL connector accepts pg_replication_bootstrap_batch_size to
control rows-per-batch during the initial CDC snapshot (default 8192,
max 1048576), but it wasn't documented in either the connector reference
or the CDC feature page.

## Reference
Verified against spiceai/spiceai trunk at f4a5089df:
- ParameterSpec: crates/data-connectors/connector-postgres/src/lib.rs:169-174
- Consumed by: crates/data-connectors/connector-postgres/src/replication.rs:436-441
- Constants: DEFAULT_BOOTSTRAP_BATCH_SIZE=8192, MAX_BOOTSTRAP_BATCH_SIZE=1048576

The parameter was added in source on 2026-05-12 (PR #10789) and is not in
any released tag, so only the unversioned vNext docs need updating.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

🔍 Pull with Spice Failed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ No banned labels detected

Failed checks:

  • ❌ Please select at least one of the required labels for this pull request: area/blog, area/docs, area/cookbook, dependencies
  • ❌ At least one assignee is required for this pull request.

Please address these issues and update your pull request.

@github-actions
Copy link
Copy Markdown

🚀 deployed to https://85453a2a.spiceai-org-website.pages.dev

@lukekim lukekim merged commit 10d5d9f into trunk May 19, 2026
6 of 10 checks passed
@lukekim lukekim deleted the fix/postgres-replication-bootstrap-batch-size branch May 19, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants