Re: pg_waldump erroneously outputs newline for FPWs, and another minor bug

Peter Geoghegan <pg@bowt.ie>

From: Peter Geoghegan <pg@bowt.ie>
To: Andres Freund <andres@anarazel.de>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>, Heikki Linnakangas <hlinnaka@iki.fi>
Date: 2019-10-29T23:42:07Z
Lists: pgsql-hackers
On Tue, Oct 29, 2019 at 4:33 PM Andres Freund <andres@anarazel.de> wrote:
> Does anybody have an opinion about fixing it just in master or also
> backpatching it? I guess there could be people having written parsers
> for the waldump output?  I'm inclined to backpatch.

The same commit from Heikki omitted one field from that record, for no
good reason. I backpatched a bugfix to the output format for nbtree
page splits a few weeks ago, fixing that problem. I agree that we
should also backpatch this bugfix.

-- 
Peter Geoghegan



Commits

  1. pg_waldump: Fix --bkp-details to not issue spurious newlines for FPWs.

  2. pg_waldump: Fix small memory leak when rmgr->rm_identify returns NULL.

  3. Revamp the WAL record format.

  4. Improve code around the recently added rm_identify rmgr callback.