On Thu, Apr 8, 2010 at 7:37 AM, Simon Riggs <simon@2ndquadrant.com> wrote:
> On Thu, 2010-04-08 at 06:58 -0400, Robert Haas wrote:
>
>> >> Thanks. Committed.
>> >
>> > Thanks. The following TODO item should be removed?
>> >
>> > "Redefine smart shutdown in standby mode to exist as soon as all
>> > read-only connections are gone."
>> > http://wiki.postgresql.org/wiki/Todo#Standby_server_mode
>
>> > Or change it to something like?
>> >
>> > "Change smart shutdown in standby mode so that it kills the startup
>> > and walreceiver process before waiting for the regular backends to die off"
>>
>> Yeah, we should do one of those two things, but I don't much care which.
>
> I do. I see no reason to do the latter, ever, so should not be added to
> any TODO.
Well, stopping recovery earlier would mean fewer locks, which would
mean a better chance for the read-only backends to finish their work
and exit quickly. But I'm not sure how much it's worth worrying
about.
...Robert