Skip to content

WPB-22958: migrate team notifications to PostGreSQL - #5660

Draft
blackheaven wants to merge 5 commits into
developfrom
gdifolco/WPB-22958-migration-postgres-team-notifications
Draft

blackheaven wants to merge 5 commits into
developfrom
gdifolco/WPB-22958-migration-postgres-team-notifications

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

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

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 26, 2026 13:18
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Oct 26, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-22958-migration-postgres-team-notifications branch 2 times, most recently from 30b07e4 to e778902 Compare October 29, 2026 19:09
@blackheaven
blackheaven force-pushed the gdifolco/WPB-22958-migration-postgres-team-notifications branch from 22d6cc3 to d6a6a22 Compare November 4, 2026 11:45
@blackheaven
blackheaven marked this pull request as draft November 5, 2026 09:20
@blackheaven

Copy link
Copy Markdown
Contributor Author

might not be migrated

@blackheaven
blackheaven force-pushed the gdifolco/WPB-22958-migration-postgres-team-notifications branch 2 times, most recently from c92b66f to f158ce2 Compare November 10, 2026 09:17
…p expired rows in copier

- new guarded migration 20261110130000-team-notifications-ttl-cleanup.sql schedules 'expire-team-notifications' hourly at :10 (pg_extension check + EXCEPTION RAISE WARNING, per the mls-proposal-refs template); supersedes the externally-provisioned-purge note in 20261026122128-team-notifications.sql
- copier full scan filters ttl(id) > 0 so rows that expired mid-migration are not copied as expired-on-arrival
- changelog documents the cron job and the fetchTeamNotifications hasMore-on-exact-fill pagination behavior change
@blackheaven
blackheaven force-pushed the gdifolco/WPB-22958-migration-postgres-team-notifications branch from f158ce2 to 706c7cb Compare November 13, 2026 19:40

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.

2 participants