Re: Re: [COMMITTERS] pgsql: Make a hard state change from catchup to streaming mode.

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Re: [COMMITTERS] pgsql: Make a hard state change from catchup to streaming mode.
Date: 2011-02-18 15:56:33
Message-ID: 1298044593.2226.4934.camel@ebony
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Fri, 2011-02-18 at 10:41 -0500, Tom Lane wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> > On Fri, Feb 18, 2011 at 10:09 AM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
> >> Make a hard state change from catchup to streaming mode.
> >> More useful state change for monitoring purposes, plus a
> >> required change for synchronous replication patch.
>
> > As far as I can see, this patch was not posted or discussed before
> > commit, and I'm not sure it's the behavior everyone wants. It has the
> > effect of preventing the system from ever going backwards from
> > "streaming" to "catchup". Is that what we want?
>
> That seems like a very bad idea from here. Being able to go back to
> catchup after loss of the streaming connection is essential for
> robustness. If we now have to restart the slave for that to happen,
> it's not an improvement.

If we lose the connection then we start another walsender process, so
that is a non-issue.

We need a hard state change at one particular point to avoid sync rep
requestors from hanging for hours when the standby is connected but a
long way from being caught up.

This was a part of my sync rep patch that it was easier to break out and
commit early. There has never been any comment or objection to this
concept and the same feature has existed in my patches for months.

--
Simon Riggs http://www.2ndQuadrant.com/books/
PostgreSQL Development, 24x7 Support, Training and Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2011-02-18 16:50:42 Re: Re: [COMMITTERS] pgsql: Make a hard state change from catchup to streaming mode.
Previous Message Andrew Dunstan 2011-02-18 15:52:15 Re: pgsql: Use $INDENT rather than indent throughout the pgindent code

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2011-02-18 16:02:22 Re: Snapshot synchronization, again...
Previous Message Magnus Hagander 2011-02-18 15:56:02 Re: Debian readline/libedit breakage