Re: Logical decoding on standby

From: Craig Ringer <craig(at)2ndquadrant(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Simon Riggs <simon(dot)riggs(at)2ndquadrant(dot)com>, Thom Brown <thom(at)linux(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Logical decoding on standby
Date: 2017-04-04 14:33:14
Views: Raw Message | Whole Thread | Download mbox
Lists: pgsql-hackers

On 4 April 2017 at 22:32, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
> Hi all
> Here's the final set of three patches on top of what's already committed.
> The first is catalog_xmin logging, which is unchanged from the prior post.
> The 2nd is support for conflict with recovery, with changes that
> should address Andres's concerns there.
> The 3rd actually enables decoding on standby. Unlike the prior
> version, no attempt is made to check the walsender configuration for
> slot use, etc. The ugly code to try to mitigate races is also removed.
> Instead, if wal_level is logical the catalog_xmin sent by
> hot_standby_feedback is now the same as the xmin if there's no local
> slot holding it down. So we're always sending a catalog_xmin in
> feedback and we should always expect to have a valid local
> oldestCatalogXmin once hot_standby_feedback kicks in. This makes the
> race in slot creation no worse than the existing race between
> hot_standby_feedback establishment and the first queries run on a
> downstream, albeit with more annoying consequences. Apps can still
> ensure a slot created on standby is guaranteed safe and conflict-free
> by having a slot on the master first.
> I'm much happier with this. I'm still fixing some issues in the tests
> for 03 and tidying them up, but 03 should allow 01 and 02 to be
> reviewed in their proper context now.

Dammit. Attached.

Craig Ringer
PostgreSQL Development, 24x7 Support, Training & Services

Attachment Content-Type Size
01-log-catalog-xmin-advances-v6.patch text/x-patch 31.2 KB
02-decoding-recovery-conflicts-v6.patch text/x-patch 20.0 KB
03-decoding-on-standby-v6.patch text/x-patch 31.5 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2017-04-04 15:39:00 Re: [PATCH] Reduce src/test/recovery verbosity
Previous Message Craig Ringer 2017-04-04 14:32:40 Re: Logical decoding on standby