Hello, The patch applies and compiles cleanly, tests pass. The code is working as well. I was able to check it by simply creating a BRIN index and filling the table with data forcing the index to request lots of work items:
create table test (a serial, b text); create index on test using brin (a) with (pages_per_range=1, autosummarize=true); insert into test select i, repeat(md5(random()::text), 30) from generate_series(1, 3000) i; LOG: could not request an autovacuum work item "BrinSummarizeRange" for "test_a_idx" LOG: could not request an autovacuum work item "BrinSummarizeRange" for "test_a_idx" ... Just couple remarks. I would rename 'requested' variable in AutoVacuumRequestWork() func to something like 'success' or 'result'. Because request is something caller does. And I would also rephrase log message as follows: request for autovacuum work item "%s" for "%s" failed Thanks! -- Ildar Musin Postgres Professional: http://www.postgrespro.com Russian Postgres Company
