Re: [PATCH] Logical decoding timeline following take II

From: Petr Jelinek <petr(at)2ndquadrant(dot)com>
To: Craig Ringer <craig(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Logical decoding timeline following take II
Date: 2016-10-24 15:49:13
Message-ID: b579759e-be05-3128-7707-ed320e90e277@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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.

>
> The test harness code will become unnecessary when proper support for
> logical failover or logical decoding on standby is added, so I'm not
> really sure it should be committed.

Yeah as I said above I split out the test harness for this reason. The
good thing is that when the followup patches get in the test harness
should be easy to removed as the changes are very localized.

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

Attachment Content-Type Size
0001-Follow-timeline-switches-in-logical-decoding.patch text/plain 13.6 KB
0002-Test-harness-for-timeline-switches-following-in-logi.patch text/plain 25.1 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2016-10-24 15:57:27 Re: Patch: Implement failover on libpq connect level.
Previous Message Albe Laurenz 2016-10-24 15:35:49 Re: Add PGDLLEXPORT to PG_FUNCTION_INFO_V1