Skip to content

feat(stovepipe): advance queue latest_request_id on ingest#323

Open
mnoah1 wants to merge 2 commits into
mnoah1/stovepipe-request-entityfrom
mnoah1/stovepipe-ingest-queue-wiring
Open

feat(stovepipe): advance queue latest_request_id on ingest#323
mnoah1 wants to merge 2 commits into
mnoah1/stovepipe-request-entityfrom
mnoah1/stovepipe-ingest-queue-wiring

Conversation

@mnoah1

@mnoah1 mnoah1 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

What?

  • After ingest inserts a new request row, advance that queue's latest request ID

Why?

Test Plan

  • Unit tests wire QueueStore expectations; integration test asserts latest_request_id on ingest

mnoah1 added 2 commits July 8, 2026 21:14
After ensureRequest succeeds, CAS-update the queue row's latest_request_id
when the request id is newer. CompareRequestID makes the advance a no-op on
dedup re-ingests; optimistic-lock retries handle concurrent ingests.
@mnoah1 mnoah1 marked this pull request as ready for review July 9, 2026 14:59
@mnoah1 mnoah1 requested review from a team, behinddwalls and sbalabanov as code owners July 9, 2026 14:59
@mnoah1 mnoah1 force-pushed the mnoah1/stovepipe-request-entity branch from bbf0abf to 90cbeef Compare July 9, 2026 18:20
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.

1 participant