Skip to content

refactor(consensus): remove namespace from client headers - #3836

Draft
numinnex wants to merge 16 commits into
masterfrom
remove_namespace_field
Draft

refactor(consensus): remove namespace from client headers#3836
numinnex wants to merge 16 commits into
masterfrom
remove_namespace_field

Conversation

@numinnex

@numinnex numinnex commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

No description provided.

numinnex and others added 13 commits August 3, 2026 12:19
Removing the namespace-exemption import left its #[cfg(debug_assertions)]
attribute orphaned onto the unconditional server_common import below it, so
every release build of the shard crate fails with 112 unresolved-name
errors. Debug builds keep the import and CI's -D warnings lane rides
clippy, which builds debug, so nothing upstream could observe it.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions github-actions Bot added the S-waiting-on-review PR is waiting on a reviewer label Aug 7, 2026
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 70.85890% with 190 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.36%. Comparing base (c272f21) to head (02b1937).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
core/shard/src/lib.rs 48.03% 51 Missing and 2 partials ⚠️
core/server-ng/src/dispatch.rs 40.00% 33 Missing ⚠️
core/simulator/src/client.rs 65.27% 25 Missing ⚠️
core/metadata/src/impls/metadata.rs 70.66% 22 Missing ⚠️
core/consensus/src/impls.rs 89.79% 5 Missing ⚠️
core/partitions/src/state_transfer.rs 16.66% 5 Missing ⚠️
core/binary_protocol/src/consensus/header.rs 94.59% 4 Missing ⚠️
core/partitions/src/iggy_partitions.rs 75.00% 4 Missing ⚠️
core/shard/src/router.rs 75.00% 4 Missing ⚠️
core/partitions/src/iggy_partition.rs 85.71% 3 Missing ⚠️
... and 27 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3836      +/-   ##
============================================
- Coverage     76.58%   76.36%   -0.22%     
  Complexity     1046     1046              
============================================
  Files          1347     1345       -2     
  Lines        171013   170418     -595     
  Branches     142367   142176     -191     
============================================
- Hits         130966   130136     -830     
- Misses        36225    36383     +158     
- Partials       3822     3899      +77     
Components Coverage Δ
Rust Core 75.60% <70.21%> (-0.18%) ⬇️
Java SDK 63.67% <ø> (ø)
C# SDK 71.13% <ø> (-1.16%) ⬇️
Python SDK 88.70% <ø> (ø)
PHP SDK 84.50% <ø> (+1.52%) ⬆️
Node SDK 96.30% <100.00%> (+0.01%) ⬆️
Go SDK 68.65% <100.00%> (-0.53%) ⬇️
Files with missing lines Coverage Δ
core/binary_protocol/src/namespace.rs 90.90% <ø> (ø)
core/binary_protocol/src/version.rs 99.22% <ø> (ø)
core/consensus/src/lib.rs 0.00% <ø> (ø)
core/consensus/src/observability.rs 40.08% <100.00%> (ø)
core/consensus/src/plane_helpers.rs 94.70% <100.00%> (-0.05%) ⬇️
core/server-ng/src/bootstrap.rs 24.89% <ø> (ø)
core/server-ng/src/http/session.rs 85.80% <ø> (ø)
core/server-ng/src/http/wire.rs 95.93% <100.00%> (ø)
core/server-ng/src/partition_reconciler.rs 92.82% <100.00%> (ø)
core/server-ng/src/responses.rs 41.27% <100.00%> (-0.05%) ⬇️
... and 56 more

... and 60 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

mmodzelewski
mmodzelewski previously approved these changes Aug 8, 2026
spetz
spetz previously approved these changes Aug 8, 2026
@numinnex
numinnex dismissed stale reviews from spetz and mmodzelewski via 02b1937 August 8, 2026 06:37
@numinnex
numinnex marked this pull request as draft August 8, 2026 06:39
@github-actions github-actions Bot removed the S-waiting-on-review PR is waiting on a reviewer label Aug 8, 2026
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.

3 participants