| 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 |
| Message-ID: | 20200509002649.rdrfaocsuyfw3dck@alap3.anarazel.de |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
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.
Greetings,
Andres Freund
| 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) |