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 |
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 |
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 |