Skip to content

Add migration to create logical replication slot for notify dashboard#4906

Open
DilwoarH wants to merge 1 commit into
mainfrom
add-ft-service-stats-db-updates-1
Open

Add migration to create logical replication slot for notify dashboard#4906
DilwoarH wants to merge 1 commit into
mainfrom
add-ft-service-stats-db-updates-1

Conversation

@DilwoarH

@DilwoarH DilwoarH commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

What

Add migration to create logical replication slot for notify dashboard

Why

This allows us to keep track of changes in the database

@DilwoarH

DilwoarH commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Needs https://github.com/alphagov/notifications-aws/pull/3035 to be merged in first

@DilwoarH DilwoarH force-pushed the add-ft-service-stats-db-updates-1 branch from 9e82a89 to 81f9491 Compare July 6, 2026 13:00
@DilwoarH DilwoarH force-pushed the add-ft-service-stats-db-updates-1 branch from 81f9491 to 661c1ca Compare July 7, 2026 09:20
@spatel033

Copy link
Copy Markdown
Contributor

max_slot_wal_keep_size is set to -1 on the local database, which means PostgreSQL will retain WAL indefinitely until a consumer connects and acknowledges them. Is there a deployment plan or any precautions in place to ensure the consumer is available and doesn't cause WAL to grow indefinitely?

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.

2 participants