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

Reply via email to