Re: Re: BUGFIX: standby disconnect can corrupt serialized reorderbuffers - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Re: BUGFIX: standby disconnect can corrupt serialized reorderbuffers
Date
Msg-id 20180306214239.ospkf6ie7aa5gm4j@alvherre.pgsql
Whole thread Raw
In response to Re: Re: BUGFIX: standby disconnect can corrupt serialized reorder buffers  (Craig Ringer <craig@2ndquadrant.com>)
List pgsql-hackers
Craig Ringer wrote:

> Revised patch attached.
> 
> I have _not_ rewritten to use sscanf yet. I'll do that next, so you can
> choose the fewer-changes patch for backpatching if desired.

Pushed, with a further change: it seems more sensible to centralize the
whole operation of building the path, rather than just the format
string, so I created a new function to do that.  The code looks cleaner
IMO this way IMO.  All tests pass in all branches.

BTW the way the XLogSegNoOffsetToRecPtr() et al macros were modified to
accept wal_segment_size at the end of the argument list, *after* the
output argument, seems pretty bad style.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: public schema default ACL
Next
From: Daniel Gustafsson
Date:
Subject: Re: [HACKERS] Support for Secure Transport SSL library on macOS as OpenSSL alternative