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(dot)musin(at)postgrespro(dot)ru