Skip to content

feat: library update after SDS-R#63

Draft
darshankabariya wants to merge 5 commits into
masterfrom
feat/library_update
Draft

feat: library update after SDS-R#63
darshankabariya wants to merge 5 commits into
masterfrom
feat/library_update

Conversation

@darshankabariya

Copy link
Copy Markdown
Contributor

why

  • SDS-R gates every repair code path on a non-empty participantId, but the C constructor has no way to set one — so through the FFI, repair is silently a no-op.
  • ReliabilityConfig is also hardcoded to defaults, leaving consumers unable to tune bloom-filter sizing, repair backoff, or response-group count for production.
  • There's also no way to remove a channel, so per-channel state grows over time.

What

  • New constructor that takes a participantId and a JSON-encoded config.
  • New SdsRemoveChannel entry point.
  • Existing constructor kept as a back-compat shim.

JSON is used for the config because it keeps the ABI forward-compatible

@darshankabariya
darshankabariya marked this pull request as draft April 25, 2026 21:44
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.

1 participant