Repository navigation
fix(collaboration): bound legacy lease reads without losing visible continuations - #181
Merged
LIghtJUNction merged 6 commits intoSep 29, 2026
Conversation
…ery effect guards Use no seek predicate for the initial page and actual cursor keys for continuations. Keep runtime-generation checks real in effect-order regression coverage, with explicit stale-generation rejection. Add PostgreSQL ordering/Unicode pagination and cloud-only regression evidence. Refs #78 #82 #83 #85
Correct the focused workflow diagnostic without bypassing failures or changing runtime source/test assertions. Preserve verified prior RED evidence and active run references in the existing progress note.
…executor error redaction
…on fixes Preserve both regression-first and PR #180 histories. Scan at most 128 rows per query, apply current target visibility per row, enforce a 4096-row/monotonic deadline budget, and return continuation only from a visible lease. Preserve existing v1 output and v2 Resource contracts. Cloud RED 36573737687: 13 cases, exactly two explicit unbounded SQL failures, no errors/skips. Runtime source SHA256 ac4247f5629403942506a80c895e4018db4ec56d3a2e279a11a6fedbbef3b74d; no unrelated handoff changes. Refs #81 #85
This was referenced Sep 29, 2026
LIghtJUNction
added a commit
that referenced
this pull request
Sep 29, 2026
…atest read integration Retain #180 f304bcf and #181 70d46ce histories without changing either branch. Work request/offer/checkpoint/proposal readers use bounded 128-row keyset scans, current row authorization/projection, 4096-candidate and monotonic-deadline budgets. Preserve their existing output/schema/state transitions. Cloud RED 36576545310 at dd441ea: 15 real PostgreSQL/signed cases, exactly five unbounded-SQL failures, no errors/skips. Artifact11038236899 ZIP SHA256 71adea8a6611c11fe1fd82f13900e0e48b224038b610ed66530bde3129198c12 independently checked with source/JUnit. Add pre-query and mid-scan deadline regressions for each work kind. Refs #78 #81 #82 #83 #85
LIghtJUNction
merged commit Sep 29, 2026
70d46ce
into
integrate/all-open-20260929
13 of 17 checks passed
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
1 participant
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.
Scope and dependency
Refs #81 #85. Narrow follow-up to #176; includes and preserves #180 head
63bafb4fdc352accd2115307f1373c28c89f92bdrather than rebuilding its changes. Merge #180 first when its exact gates pass; this PR then reduces to the collaboration slice. Do not close the broad acceptance issues.Head 70d46ce, tree 6fb1502404967778e6e533faf414866557b80a3b.
communication.lease_list@1previously materialized every lease for the holder before applying limit or current visibility. Read at most128 SQL rows per batch, scan using actual last-seen IDs, retain current target authorization for every candidate, and look ahead for a visible item before returning next_after.Cloud evidence
RED run 36573737687 at test-only headfd4cde5809e44215d9687858429ea697baae436d/tree3a28244429f0e6e5ff3a7bf0fc531ac6411a531e: 13 cases, exactly2 failures,0 errors,0 skips; both failures independently capture the exact missing-LIMIT SQL, not merely an internal_error. Downloaded artifact11036112969 SHA256 84d1745fdd62f383396479358293752bbb9582d5b4073e3b6994110b158b0ffe verified with source/JUnit. Other11 existing collaboration tests pass.
Current focused run 36574509686 has been triggered; result and final combined full gates are not yet claimed GREEN. Preserve the complete four-shard exact node-ID gate, conformance/tokenizer, package builds and applicable specialist checks on the final source.
Runtime source is exactly blobf978dc61c866d1c48fab2e3a87979b67d7a8273e/SHA256ac4247f5629403942506a80c895e4018db4ec56d3a2e279a11a6fedbbef3b74d. Auxiliary source-only preparation publishes a single hash-constrained blob, never changes refs and is absent from the product tree. All project execution occurs in cloud CI.
Live progress note: #81 comment5890957893. No production deployment, Root/PIN, real funds, external delivery or recovery promotion.