Re: Include sequence relation support in logical replication

From: Andres Freund <andres(at)anarazel(dot)de>
To: Cary Huang <cary(dot)huang(at)highgo(dot)ca>
Cc: Craig Ringer <craig(at)2ndquadrant(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Include sequence relation support in logical replication
Date: 2020-05-09 00:26:49
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers


On 2020-05-08 16:32:38 -0700, Cary Huang wrote:
> I have added more regression test cases to the sequence replication
> patch with emphasis on transactions and rollback per your
> suggestions. I find that when a transaction is aborted with rollback,
> the decoder plugin will not receive the change but the sequence value
> will in fact advance if nextval() or setval() were called.

Right. The sequence advances shouldn't be treated
transactionally. That's already (optionally) done similarly for
messages, so you should be able to copy that code.


Andres Freund

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2020-05-09 00:49:20 Re: stat() on Windows might cause error if target file is larger than 4GB
Previous Message Tomas Vondra 2020-05-09 00:10:58 Re: [PATCH] Fix division by zero (explain.c)