Re: Improve logical replication usability when tables lack primary keys

From: "Euler Taveira" <euler(at)eulerto(dot)com>
To: "Chao Li" <li(dot)evan(dot)chao(at)gmail(dot)com>, "Postgres hackers" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Improve logical replication usability when tables lack primary keys
Date: 2025-11-11 12:09:02
Message-ID: ca7182b5-6817-4a74-802d-c3756f64f514@app.fastmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Nov 10, 2025, at 5:06 AM, Chao Li wrote:
> I evaluated a few approaches and am proposing the following:
>
> - Introduce a new GUC: `logical_replication_fallback_to_full_identity`.
> - When enabled, if a table being logically replicated has no primary
> key, the system automatically uses `REPLICATION IDENTITY FULL` for that
> table.
> - This setting can be applied at the database level, so large systems
> do not need to enable it cluster-wide unless desired.
> - When the WAL sender transmits relation metadata, if fallback has
> occurred, it explicitly reports `FULL` as the replication identity to
> the subscriber, so there is limited impact on the subscriber.
>

If I understand your proposal correctly, you want to add a new fallback to
replica identity. We already have a fallback for DEFAULT that means no primary
key is the same as NOTHING. I didn't like your proposal. It is too restrictive.

However, I see some usefulness in introducing a GUC default_replica_identity.
The proposal is similar to access method (default_table_access_method). The
DEFAULT option selects the replica identity sets as default_replica_identity
parameter. You need to add a new option (PRIMARY KEY); that should be the
default value. (If we don't want to break the backward compatibility, this new
option should fallback to NOTHING if there is no primary key. Another
alternative is to have a strict and non-strict option. I prefer the former.) Of
course, the USING INDEX option cannot be used. For pg_dump, you need to use SET
command to inform the default_replica_identity value so tables with the same
option as default_replica_identity doesn't emit an ALTER TABLE command.

--
Euler Taveira
EDB https://www.enterprisedb.com/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Álvaro Herrera 2025-11-11 12:22:23 Re: [PATCH} Move instrumentation structs
Previous Message Matthias van de Meent 2025-11-11 12:06:30 Re: [WiP] B-tree page merge during vacuum to reduce index bloat