Re: [COMMITTERS] pgsql: Fix pg_receivexlog --synchronous

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Simon Riggs <simon(at)2ndquadrant(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Fix pg_receivexlog --synchronous
Date: 2016-08-29 12:16:36
Message-ID: CAB7nPqRb94jehwAGY-1bDKLN_-Uo=S+OZejmo41_5fEricd3GQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Mon, Aug 29, 2016 at 8:34 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Simon Riggs <simon(at)2ndQuadrant(dot)com> writes:
>> Fix pg_receivexlog --synchronous
>
> The buildfarm says you broke the 9.5 branch.
>
> In general, pushing inessential patches just a few hours before a wrap
> deadline is a dangerous business. Pushing them without any testing
> is close to irresponsible.

This area of the code has faced some refactoring from Magnus lately,
so you need this on REL9_5_STABLE:
--- a/src/bin/pg_basebackup/receivelog.c
+++ b/src/bin/pg_basebackup/receivelog.c
@@ -534,7 +534,7 @@ ReceiveXlogStream(PGconn *conn, XLogRecPtr
startpos, uint32 timeline,
}
else
{
- if (stream->synchronous)
+ if (synchronous)
reportFlushPosition = true;
else
reportFlushPosition = false;
--
Michael

Attachment Content-Type Size
fix-receivexlog-95.patch application/x-download 423 bytes

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2016-08-29 12:57:41 pgsql: Doc: improve 9.6 description of SP-GiST traverse values.
Previous Message Tom Lane 2016-08-29 11:34:52 Re: [COMMITTERS] pgsql: Fix pg_receivexlog --synchronous

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2016-08-29 12:28:35 Re: Renaming of pg_xlog and pg_clog
Previous Message Robert Haas 2016-08-29 11:53:38 Re: Declarative partitioning - another take