On Tue, May 11, 2021 at 6:31 PM Justin Pryzby <pry...@telsasoft.com> wrote: > > On Tue, May 11, 2021 at 05:37:50PM +0530, Bharath Rupireddy wrote: > > 3) Should the Assert(nindexes > 0); in begin_parallel_vacuum just be > > Assert(nindexes > 1); as this function is entered only when indexes > > are > 1? > > I think you're right, at least with the current implementation that > parallelization is done across indexes. Same in parallel_vacuum_main. >
Yeah, as code stands both of you are right. However, it can be helpful to test parallelism even with one index say if we implement something like force_parallel_mode = regress or parallel_leader_participation = off. -- With Regards, Amit Kapila.
