On Tue, Oct 13, 2020 at 7:26 PM Noah Misch <n...@leadboat.com> wrote: > 1. Disable parallelism for the index build under ExecuteTruncateGuts(). > Nobody will mourn a performance loss from declining parallelism for an > empty index, but I feel like this is fixing in the wrong place. > 2. Make _bt_begin_parallel() and begin_parallel_vacuum() recognize the > debug_query_string==NULL case and reproduce it on the worker. > 3. Require bgworkers to set debug_query_string before entering code of vacuum, > truncate, etc. Logical replication might synthesize a DDL statement, or it > might just use a constant string. > > I tend to prefer (2), but (3) isn't bad. Opinions?
I also prefer 2. Thanks -- Peter Geoghegan