Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Fix test not detecting tables with missing primary keys and missing replica identities, then add more replica identities.#16647

Merged
reivilibre merged 3 commits into
developfrom
rei/replica_identities_2
Nov 16, 2023
Merged

Fix test not detecting tables with missing primary keys and missing replica identities, then add more replica identities.#16647
reivilibre merged 3 commits into
developfrom
rei/replica_identities_2

Conversation

@reivilibre
Copy link
Copy Markdown
Contributor

Fixes: #16224

Follows: #16456

Base: develop

Original commit schedule, with full messages:

  1. Fix the CI query that did not detect all cases of missing primary keys

  2. Add more missing REPLICA IDENTITY entries

@reivilibre reivilibre requested a review from a team as a code owner November 16, 2023 11:40
@reivilibre reivilibre merged commit 830988a into develop Nov 16, 2023
@reivilibre reivilibre deleted the rei/replica_identities_2 branch November 16, 2023 12:26
erikjohnston added a commit that referenced this pull request Nov 16, 2023
…issing replica identities, then add more replica identities. (#16647)"

This reverts commit 830988a.
@erikjohnston erikjohnston mentioned this pull request Nov 16, 2023
erikjohnston added a commit that referenced this pull request Nov 16, 2023
…issing replica identities, then add more replica identities. (#16647)" (#16652)

This reverts commit 830988a.
erikjohnston added a commit that referenced this pull request Nov 16, 2023
…issing replica identities, then add more replica identities. (#16647)"

This reverts commit 830988a.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

All tables in Postgres should have a REPLICA IDENTITY available so that Postgres logical replication can be used

2 participants