Skip to content

WPB-29575: migrate user key store to PostGreSQL - #5642

Open
blackheaven wants to merge 3 commits into
developfrom
gdifolco/WPB-29575-migration-postgres-user-keys
Open

blackheaven wants to merge 3 commits into
developfrom
gdifolco/WPB-29575-migration-postgres-user-keys

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-29575

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@blackheaven
blackheaven requested review from a team as code owners October 22, 2026 19:48
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Oct 22, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-29575-migration-postgres-user-keys branch 2 times, most recently from 32c7112 to 5e86f91 Compare November 4, 2026 13:52
@blackheaven
blackheaven force-pushed the gdifolco/WPB-29575-migration-postgres-user-keys branch 2 times, most recently from d177495 to 2c00faf Compare November 10, 2026 07:54
…claimKeyImpl

- Copy loop (Migration.hs) now calls Postgres.insertUserKeyIfAbsent
  (ON CONFLICT (key) DO NOTHING): a stale Cassandra snapshot must never
  clobber a newer Postgres row, e.g. a key reclaim after the row was
  copied. The dual-write mirror keeps ON CONFLICT DO UPDATE because it
  mirrors the unconditional Cassandra overwrite of insertKeyImpl.
- DualWrite.ClaimKey delegates to the Cassandra claimKeyImpl instead of
  re-implementing its read-then-write, then mirrors the insert.
@blackheaven
blackheaven force-pushed the gdifolco/WPB-29575-migration-postgres-user-keys branch from 4c3f4b3 to 3f23f71 Compare November 13, 2026 11:13

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants