Re: [PATCH] Logical decoding timeline following take II

On 24 October 2016 at 23:49, Petr Jelinek <petr(at)2ndquadrant(dot)com> wrote:
> Hi Craig,
>
> On 01/09/16 06:08, Craig Ringer wrote:
>> Hi all
>>
>> Attached is a rebased and updated logical decoding timeline following
>> patch for 10.0.
>>
>> This is a pre-requisite for the pending work on logical decoding on
>> standby servers and simplified failover of logical decoding.
>>
>
> I went over this and it looks fine to me, I only rebased the patch on
> top of current version (we renamed pg_xlog which broke the tests) and
> split the test harness to separate patch. Otherwise I would consider
> this to be ready for committer.
>
> I think this should go in early so that there is enough time in the
> cycle to uncover potential issues if there are any, even though it looks
> all correct to me.

Thanks for the review and update.

--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services