From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Relax overly strict assertion |
Date: | 2019-02-12 21:59:25 |
Message-ID: | E1gtg5N-0005Rd-AF@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Relax overly strict assertion
Ever since its birth, ReorderBufferBuildTupleCidHash() has contained an
assertion that a catalog tuple cannot change Cmax after acquiring one. But
that's wrong: if a subtransaction executes DDL that affects that catalog
tuple, and later aborts and another DDL affects the same tuple, it will
change Cmax. Relax the assertion to merely verify that the Cmax remains
valid and monotonically increasing, instead.
Add a test that tickles the relevant code.
Diagnosed by, and initial patch submitted by: Arseny Sher
Co-authored-by: Arseny Sher
Discussion: https://postgr.es/m/874l9p8hyw.fsf@ars-thinkpad
Branch
------
REL9_5_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/45ed482f1a899dbf5b0db13686d817718ae8cf97
Modified Files
--------------
contrib/test_decoding/expected/ddl.out | 18 ++++++++++++++++++
contrib/test_decoding/sql/ddl.sql | 13 +++++++++++++
src/backend/replication/logical/reorderbuffer.c | 14 +++++++++-----
3 files changed, 40 insertions(+), 5 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | pgsql | 2019-02-12 23:08:24 | pgsql: Tag refs/tags/REL9_4_21 was created |
Previous Message | Alvaro Herrera | 2019-02-12 21:32:25 | Re: pgsql: Use Getopt::Long for catalog scripts |