On Thur, Nov 11, 2021 10:41 AM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
> I've attached a draft patch that refactors parallel vacuum and separates
> parallel-vacuum-related code to new file vacuumparallel.c.
> After discussion, I'll divide the patch into logical chunks.
Hi.
I noticed few minor issues in the patch.
1)
+ /*
+ * Parallel unsafe indexes can be processed only by leader (these are
+ * processed in lazy_serial_process_indexes() by leader.
+ */
It seems the function name in the comments should be serial_vacuum_unsafe_indexes
2)
+ stats->parallel_workers_can_process =
+ index_parallel_vacuum_is_safe(pvc->indrels[i],
+ pvc->num_index_scans,
+ bulkdel);
The function index_parallel_vacuum_is_safe also return false for the
index < min_parallel_index_scan_size cutoff which seems parallel safe. So,
maybe we can rename the function to xxx_worker_can_process() ?
Best regards,
Hou zj