Postgres Pro
Downloads
Home   >   mailing lists

Re: [PATCH] Race condition in logical walsender causes longpostgresql shutdown delay - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [PATCH] Race condition in logical walsender causes longpostgresql shutdown delay
Date
Msg-id 20190927012213.GA10253@alvherre.pgsql
Whole thread Raw
In response to Re: [PATCH] Race condition in logical walsender causes longpostgresql shutdown delay  (Jeff Janes <jeff.janes@gmail.com>)
Responses Re: [PATCH] Race condition in logical walsender causes longpostgresql shutdown delay
List pgsql-hackers
Tree view 
On 2019-Sep-26, Jeff Janes wrote:

> On Wed, Sep 11, 2019 at 3:52 PM Alvaro Herrera <alvherre@2ndquadrant.com>
> wrote:
> 
> > Reading over this code, I noticed that the detection of the catch-up
> > state ends up being duplicate code, so I would rework that function as
> > in the attached patch.
> >
> > The naming of those flags (got_SIGUSR2, got_STOPPING) is terrible, but
> > I'm not going to change that in a backpatchable bug fix.
> 
> Hi Alvaro, does this count as a review?

Well, I'm already a second pair of eyes for Craig's code, so I think it
does :-)  I would have liked confirmation from Craig that my change
looks okay to him too, but maybe we'll have to go without that.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

pgsql-hackers by date:

Previous
From: Kyotaro Horiguchi
Date:
Subject: Re: FETCH FIRST clause PERCENT option
Next
From: Michael Paquier
Date:
Subject: Re: PostgreSQL12 and older versions of OpenSSL