|From:||Stephen Frost <sfrost(at)snowman(dot)net>|
|To:||David Steele <david(at)pgmasters(dot)net>|
|Cc:||Nathan Bossart <nathandbossart(at)gmail(dot)com>, Chapman Flack <chap(at)anastigmatix(dot)net>, Magnus Hagander <magnus(at)hagander(dot)net>, pgsql-hackers(at)lists(dot)postgresql(dot)org|
|Subject:||Re: Postgres restart in the middle of exclusive backup and the presence of backup_label file|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
* David Steele (david(at)pgmasters(dot)net) wrote:
> On 4/4/22 11:42 AM, Nathan Bossart wrote:
> >I noticed a couple of other things that can be removed. Since we no longer
> >wait on exclusive backup mode during smart shutdown, we can change
> >connsAllowed (in postmaster.c) to a boolean and remove CAC_SUPERUSER. We
> >can also remove a couple of related notes in the documentation. I've done
> >all this in the attached patch.
> These changes look good to me. IMV it is a real bonus how much the state
> machine has been simplified.
> I've also run this patch through the pgbackrest regression tests without any
Please find attached an updated patch + commit message. Mostly, I just
went through and did a bit more in terms of updating the documentation
and improving the comments (there were some places that were still
worrying about the chance of a 'stray' backup_label file existing, which
isn't possible any longer), along with some additional testing and
review. This is looking pretty good to me, but other thoughts are
certainly welcome. Otherwise, I'm hoping to commit this tomorrow.
|Next Message||Robert Haas||2022-04-05 15:27:09||Re: Proposal for internal Numeric to Uint64 conversion function.|
|Previous Message||Robert Haas||2022-04-05 15:05:46||Re: Mark all GUC variable as PGDLLIMPORT|