Tom Lane <t...@sss.pgh.pa.us> writes:
> Thomas Munro <thomas.mu...@gmail.com> writes: >> On Fri, Aug 14, 2020 at 4:45 AM Tom Lane <t...@sss.pgh.pa.us> wrote: >>> After some more rethinking and testing, here's a v5 that feels >>> fairly final to me. I realized that the logic in canAcceptConnections >>> was kind of backwards: it's better to check the main pmState restrictions >>> first and then the smart-shutdown restrictions afterwards. > >> LGTM. I tested this a bit today and it did what I expected for >> parallel queries and vacuum, on primary and standby. > > Thanks for reviewing! I'll do the back-patching and push this today. FWIW, I've also looked through the patch and it's fine. Moderate testing also found no issues, check-world works, bgws are started during smart shutdown as expected. And surely this is better than the inital shorthack of allowing only parallel workers.
