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
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 |
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 |