Skip to content

[3/N Status & List APIs] feat(storage): add request read-model stores#322

Open
albertywu wants to merge 1 commit into
wua/status-list-read-model-schemafrom
wua/status-list-read-model-storage
Open

[3/N Status & List APIs] feat(storage): add request read-model stores#322
albertywu wants to merge 1 commit into
wua/status-list-read-model-schemafrom
wua/status-list-read-model-storage

Conversation

@albertywu

@albertywu albertywu commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Add mechanical storage contracts and MySQL implementations for request summaries, queue-ordered summaries, and change URI mappings.

Test Plan

make fmt && make build && make test && make check-mocks && make e2e-test

Stack

  1. [1/N Status & List APIs] docs: RFC for Status and List APIs #314
  2. [2/N Status & List APIs] feat(storage): add request read-model schema #321
  3. @ [3/N Status & List APIs] feat(storage): add request read-model stores #322

Summary:
Add mechanical storage contracts and MySQL implementations for authoritative request summaries, queue-ordered summaries, and change URI mappings.

Keep retry identity, conflict reconciliation, and projection decisions in callers while covering duplicate, CAS, ordering, cursor, and JSON normalization behavior in the shared storage contract suite.

Test Plan:
make fmt
make mocks
make gazelle
./tool/bazel test //test/integration/submitqueue/extension/storage/mysql:go_default_test --test_output=errors

Revert Plan:
Revert this commit. No production callers use the new stores yet.

API Changes:
None.

Monitoring and Alerts:
Storage operation metrics use the existing extension metric pattern.
@albertywu albertywu changed the title feat(storage): add request read-model stores [3/N Status & List APIs] feat(storage): add request read-model stores Jul 9, 2026
@albertywu albertywu marked this pull request as ready for review July 9, 2026 01:40
@albertywu albertywu requested review from a team, behinddwalls and sbalabanov as code owners July 9, 2026 01:40
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