pgsql: Fix assertion during streaming of multi-insert toast changes.

From: Amit Kapila <akapila(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix assertion during streaming of multi-insert toast changes.
Date: 2021-05-27 02:39:34
Message-ID: E1lm5vq-0006lc-F6@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix assertion during streaming of multi-insert toast changes.

While decoding the multi-insert WAL we can't clean the toast untill we get
the last insert of that WAL record. Now if we stream the changes before we
get the last change, the memory for toast chunks won't be released and we
expect the txn to have streamed all changes after streaming. This
restriction is mainly to ensure the correctness of streamed transactions
and it doesn't seem worth uplifting such a restriction just to allow this
case because anyway we will stream the transaction once such an insert is
complete.

Previously we were using two different flags (one for toast tuples and
another for speculative inserts) to indicate partial changes. Now instead
we replaced both of them with a single flag to indicate partial changes.

Reported-by: Pavan Deolasee
Author: Dilip Kumar
Reviewed-by: Pavan Deolasee, Amit Kapila
Discussion: https://postgr.es/m/CABOikdN-_858zojYN-2tNcHiVTw-nhxPwoQS4quExeweQfG1Ug@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/6f4bdf81529fdaf6744875b0be99ecb9bfb3b7e0

Modified Files
--------------
contrib/test_decoding/expected/stream.out | 24 +++++++++++++
contrib/test_decoding/sql/stream.sql | 18 ++++++++++
src/backend/replication/logical/reorderbuffer.c | 46 ++++++++++++++-----------
src/include/replication/reorderbuffer.h | 27 ++++-----------
4 files changed, 73 insertions(+), 42 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2021-05-27 05:58:43 pgsql: doc: Fix description of some GUCs in docs and postgresql.conf.sa
Previous Message Michael Paquier 2021-05-26 10:54:46 pgsql: Fix typo in heapam.c