pgsql: Fix logical decoding bug leading to inefficient reopening of fil - Mailing list pgsql-committers

From Andres Freund
Subject pgsql: Fix logical decoding bug leading to inefficient reopening of fil
Date
Msg-id E1ZCQpw-0006MW-PZ@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix logical decoding bug leading to inefficient reopening of files.

When spilling transaction data to disk a simple typo caused the output
file to be closed and reopened for every serialized change. That happens
to not have a huge impact on linux, which is why it probably wasn't
noticed so far, but on windows that appears to trigger actual disk
writes after every change. Not fun.

The bug fortunately does not have any impact besides speed. A change
could end up being in the wrong segment (last instead of next), but
since we read all files to the end, that's just ugly, not really
problematic. It's not a problem to upgrade, since transaction spill
files do not persist across restarts.

Bug: #13484
Reported-By: Olivier Gosseaume
Discussion: 20150703090217.1190.63940@wrigleys.postgresql.org

Backpatch to 9.4, where logical decoding was added.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/b2f6f749c7a5936adbb555e248e8e4df35c00a4a

Modified Files
--------------
src/backend/replication/logical/reorderbuffer.c |    2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Joe Conway
Date:
Subject: pgsql: Make RLS related error messages more consistent and compliant.
Next
From: Andres Freund
Date:
Subject: pgsql: Fix pg_recvlogical not to fsync output when it's a tty or pipe.