Skip to content
This repository was archived by the owner on Jun 19, 2026. It is now read-only.

receive: Disable shuffle sharding for default tenant#312

Merged
yuchen-db merged 1 commit into
db_mainfrom
yuchen-db/default-tenant-no-shuffle-sharding
Mar 14, 2026
Merged

receive: Disable shuffle sharding for default tenant#312
yuchen-db merged 1 commit into
db_mainfrom
yuchen-db/default-tenant-no-shuffle-sharding

Conversation

@yuchen-db

Copy link
Copy Markdown
Collaborator

The default tenant (catch-all for requests without a tenant header) now writes to ALL endpoints instead of a sharded subset. The default tenant ID is threaded from the CLI flag through the hashring construction chain, and getTenantShardCached short-circuits to return the base ring when the tenant matches the default tenant ID.

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Verification

The default tenant (catch-all for requests without a tenant header)
now writes to ALL endpoints instead of a sharded subset, preventing
silent data loss when unattributed traffic arrives. The default tenant
ID is threaded from the CLI flag through the hashring construction
chain, and getTenantShardCached short-circuits to return the base ring
when the tenant matches the default tenant ID.

Co-authored-by: Isaac
@yuchen-db yuchen-db requested review from mkusumdb and willh-db March 13, 2026 23:37
@yuchen-db yuchen-db merged commit e85e300 into db_main Mar 14, 2026
23 of 27 checks passed
@yuchen-db yuchen-db deleted the yuchen-db/default-tenant-no-shuffle-sharding branch March 14, 2026 00:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants