Skip to content

Add replication destination prefix support#247

Merged
atimin merged 2 commits into
reductstore:mainfrom
ychampion:feat/246-replica-prefix
Jul 13, 2026
Merged

Add replication destination prefix support#247
atimin merged 2 commits into
reductstore:mainfrom
ychampion:feat/246-replica-prefix

Conversation

@ychampion

@ychampion ychampion commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Closes #246

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • CHANGELOG.md has been updated (for bug fixes / features / docs)

What kind of change does this PR introduce?

Feature.

What was changed?

  • Add --prefix to replica create and update commands.
  • Preserve the current prefix when an update omits the option and show the configured prefix in replica show.
  • Align replica commands with API v1.21 and the prefix-capable reduct-rs revision.

Related issues

Does this PR introduce a breaking change?

Yes. Replica create and update no longer expose the deprecated --each-s option removed from API v1.21. Other commands that use --each-s are unchanged.

Other information:

Validation performed:

  • RS_API_TOKEN=TOKEN cargo test --locked -- --test-threads=1 (192 passed against reduct/store:main)
  • cargo build --locked
  • cargo fmt --all -- --check
  • cargo package --locked --allow-dirty --no-verify
  • Built CLI create, show, and update smoke with prefix readback

Constraint: API 1.21 exposes destination prefixes and removes the deprecated replication each_s field.
Rejected: Keep reduct-rs 1.20 from crates.io | its replication model cannot serialize the new prefix.
Confidence: high
Scope-risk: moderate
Directive: Return reduct-rs to the registry release after the API 1.21 SDK is published.
Tested: 192 locked tests against reduct/store:main; cargo build --locked; cargo fmt --all -- --check; cargo package --locked --allow-dirty --no-verify; built CLI create/show/update smoke
Not-tested: Cross-target release builds; strict Clippy is already non-clean across untouched code

@atimin atimin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @ychampion ,

thank you very much for your contribution! The PR is perfect, just update docs as I mentioned in the change request.

Thank you!

Comment thread CHANGELOG.md
Comment thread README.md Outdated
@reductstore reductstore deleted a comment from chatgpt-codex-connector Bot Jul 13, 2026

@atimin atimin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perfect! Thank you!

@atimin atimin merged commit f0739ab into reductstore:main Jul 13, 2026
10 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.

CLI: support replication prefix setting

2 participants