Re: Logical decoding on standby

Rebased series attached, on top of current master (which includes
logical replicaiton).

I'm inclined to think I should split out a few of the changes from
0005, roughly along the lines of the bullet points in its commit
message. Anyone feel strongly about how granular this should be?

This patch series is a pre-requisite for supporting logical
replication using a physical standby as a source, but does not its
self enable logical replication from a physical standby.

On 23 January 2017 at 23:03, Thom Brown <thom(at)linux(dot)com> wrote:
> On 5 January 2017 at 01:21, 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.
>
> Patch 5 no longer applies:
>
> patching file src/include/pgstat.h
> Hunk #1 FAILED at 745.
> 1 out of 1 hunk FAILED -- saving rejects to file src/include/pgstat.h.rej
>
> --- src/include/pgstat.h
> +++ src/include/pgstat.h
> @@ -745,7 +745,8 @@ typedef enum
> WAIT_EVENT_SYSLOGGER_MAIN,
> WAIT_EVENT_WAL_RECEIVER_MAIN,
> WAIT_EVENT_WAL_SENDER_MAIN,
> - WAIT_EVENT_WAL_WRITER_MAIN
> + WAIT_EVENT_WAL_WRITER_MAIN,
> + WAIT_EVENT_STANDBY_LOGICAL_SLOT_CREATE
> } WaitEventActivity;
>
> /* ----------
>
> Could you rebase?
>
> Thanks
>
> Thom

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