Repository navigation
D-STO-1 (resident half): owner-stamped batches over resident lanes - #22
Merged
Merged
Conversation
hubspo-store::table::Table holds one dataset's records as resident lanes with a live plane and one validity plane per nullable lane; reads borrow them. A Batch (inserts, updates, deletes) is validated whole and lands as exactly one version or not at all, only for the table's owner (MailboxId, write on behalf). apply() returns the plane of touched rows, which is what list::refresh takes. Ordinals are never reused. Tests: 60 random batches match a plain model, lanes and planes alike; twelve refusal kinds leave the table untouched; a list maintained from the store's changed planes equals a full recomputation over 50 batches. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
AdaWorldAPI
pushed a commit
that referenced
this pull request
Oct 14, 2026
…#22-#25 D-QRY-1 points at the D-CMR set algebra (relalg prototype, #1546), D-QRY-5 at the D-CMR-5 preimage hop, D-FLOW-1 at graph-flow-lance. New ask ISS-LG-GRAPHFLOW-LANCE-PATHS; the preflight pin issue is still open at c21bd165. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
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.
What
hubspo-store::table::Tableholds one dataset (tenant × object type, plan §5.5) in memory.U32/I32lanes, a live plane (plane 0), and one validity plane per nullable lane.planes()lends them to mask-risc; no read copies a lane.Batch(inserts, updates, deletes) is validated as a whole and lands as exactly one new version, or not at all.MailboxId; write on behalf, one writer per mailbox).apply()returnsApplied { version, first_insert, changed }.changedis the plane thatlist::refresh(D-QRY-2: active / static lists and membership history as mask programs #20) takes.Persistence, one Lance version per published batch using P-SPARSE-1's base + sparse delta, is the other half of D-STO-1 and comes next. It will sit behind a feature so this crate's default CI stays fast.
Evidence
hubspo-store/tests/table.rs:hubspo-query/tests/store_list.rsruns end to end: catalog → filter compiler →list::refresh, fed by the store's ownchangedplane. Over 50 batches it equals a full recomputation.Disable runs; every one went red:
changed(red in the store test and in the list test);Workspace fmt, clippy (
--all-targets --no-deps -D warnings), tests andhubspo-guardsare green locally. No serde in zone 2. Rebased onmainafter #21; no other HubSPO-rs PR is open.🤖 Generated with Claude Code
https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
Generated by Claude Code