Hi,autovac_get_workitem_name() declaration seems redundant and should be removed. The same thing with including "utils/lsyscache.h" in brin.c.
The 'requested' variable in brininsert() I would again rename to something like 'success' because a work item is requested anyway but what matters is whether the request was satisfied/successful.
Except for those two points everything is fine and works as expected. -- Ildar Musin i.mu...@postgrespro.ru