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 problems.
Regards, -- -David da...@pgmasters.net