Reduce forwarded owner lookup and qualify write capacity - #24
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Measured evidence
DescribeandQuery, and three reads for a local query that still verifies catalog and authority. End-to-end latency for the combined client path has not yet been measured.Verification
Focused adapter, application, parser, follower recovery, and LTX tests; workspace format/Clippy; architecture, module layout, documentation, and SQL/peer checks passed on the implementation revisions. The runtime suite, counted-store routing test, workspace all-feature check and Clippy, API docs, boundaries, module layout, document gates, and SQL/peer contract checks pass at the latest revision. All seven CI checks passed at the previous revision; the new CI run must pass before marking this PR ready.
Follow-up work outside this PR
Publication optimization needs a controlled A/A baseline, root-preparation subphase attribution, and a numeric improvement gate. Follower transport optimization needs attribution of member resolution, TCP, authority lookup, and follower fsync. These are future changes; this PR provides the measurement and recovery evidence to guide them.
Local forwarded-client verification
DescribeplusQuerymakes zero entry-node metadata body reads; a local query still makes three exact reads. New regression checks refuse a stale peer handle after drain and suppress an in-flight query result after node-lease fencing.runtime_with_peer. With a synthetic 2 ms delay per object read, the full signed loopback query p95 fell from 30–35 ms to 22–26 ms at concurrency 16; the routing-only p95 fell from 20–21 ms to about 5.5 ms. Entry-node metadata reads fell from three per call to zero.crabbuild-target/.../evidencedirectory.cargo test -p cellule-runtime --features test-support --locked -- --test-threads=1passed, including 379 unit and 161 runtime integration tests; the 18 protocol and 18 peer HTTP tests passed. A parallel ownership run had one intermittent shutdown-result assertion failure after lease fencing; that exact test passed twice alone and the serialized ownership suite passed. The new read-output fence test passed. Formatting, focused Clippy, boundaries, and module layout passed.