Logflare / logflare / 3823b29da3249b2d2ce61f1205c3389b2f41d961-PR-3617

81%

main: 82%

Build Type

Pull #3617

github

Committed by djwhitt

Commit Message

fix(ingest queue): reject claim when row deleted mid-claim

claim_pending/2 bumped the claim counter then ignored update_element's
result, so a row deleted between the counter bump and the status write
was still reported as claimed — surfacing downstream as a spurious
missing/failed message. Return update_element's result instead, which is
false for a vanished key, restoring the old select_replace semantics
(a row deleted in the claim window is not claimed).

Also clarify the concurrent-claim test comment: it passes
deterministically for correct code; the race setup only exists to make a
regression actually fail.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #3617: perf(ingest): claim pending events via update_counter CAS-lite

Coverage Stats

16 of 17 new or added lines in 3 files covered. (94.12%)

21 existing lines in 2 files now uncovered.

13226 of 16293 relevant lines covered (81.18%)

5083.95 hits per line