Nathan Bossart <nathandbossart@gmail.com> writes:
> On Wed, Dec 28, 2022 at 03:13:23PM -0500, Tom Lane wrote:
>> + executeCommand(conn, "VACUUM (ONLY_DATABASE_STATS);", echo);
> When I looked at this, I thought it would be better to send the command
> through the parallel slot machinery so that failures would use the same
> code path as the rest of the VACUUM commands. However, you also need to
> adjust ParallelSlotsWaitCompletion() to mark the slots as idle so that the
> slot array can be reused after it is called.
Hm. I was just copying the way commands are issued further up in the
same function. But I think you're right: once we've done
ParallelSlotsAdoptConn(sa, conn);
it's probably not entirely kosher to use the conn directly.
regards, tom lane