From: | Craig Ringer <craig(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Simon Riggs <simon(dot)riggs(at)2ndquadrant(dot)com> |
Subject: | Re: Logical decoding on standby |
Date: | 2017-01-06 04:07:39 |
Message-ID: | CAMsr+YGXE2rPBX-Auv=Y4owBp8NgqiNKWW4Ly+gcszV7n5HjAA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 5 January 2017 at 13:12, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
> On Thu, Jan 5, 2017 at 10:21 AM, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
>> On 5 January 2017 at 09:19, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
>>
>>> so here's a rebased series on top of master. No other changes.
>>
>> Now with actual patches.
>
> Looking at the PostgresNode code in 0001...
> +=pod $node->pg_recvlogical_upto(self, dbname, slot_name, endpos,
> timeout_secs, ...)
> +
> This format is incorrect. I think that you also need to fix the
> brackets for the do{} and the eval{] blocks.
>
> + push @cmd, '--endpos', $endpos if ($endpos);
> endpos should be made a mandatory argument. If it is not defined that
> would make the test calling this routine being stuck.
> --
> Michael
Acknowledged and agreed. I'll fix both in the next revision. I'm
currently working on hot standby replies, but will return to this
ASAP.
--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Craig Ringer | 2017-01-06 04:14:34 | Re: proposal: session server side variables |
Previous Message | Jim Nasby | 2017-01-06 03:50:52 | Re: Faster methods for getting SPI results |