On Sun, Jun 17, 2018 at 9:39 PM, Andrey V. Lepikhov <a.lepik...@postgrespro.ru> wrote: > Patch '0001-retail-indextuple-deletion' introduce new function > amtargetdelete() in access method interface. Patch > '0002-quick-vacuum-strategy' implements this function for an alternative > strategy of lazy index vacuum, called 'Quick Vacuum'.
My compiler shows the following warnings: /code/postgresql/root/build/../source/src/backend/access/nbtree/nbtree.c: In function ‘bttargetdelete’: /code/postgresql/root/build/../source/src/backend/access/nbtree/nbtree.c:1053:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] if (needLock) ^~ /code/postgresql/root/build/../source/src/backend/access/nbtree/nbtree.c:1055:4: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’ npages = RelationGetNumberOfBlocks(irel); ^~~~~~ /code/postgresql/root/build/../source/src/backend/access/nbtree/nbtree.c:1074:3: warning: ‘blkno’ may be used uninitialized in this function [-Wmaybe-uninitialized] cleanup_block(info, stats, blkno); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ I think that they're both harmless, though. -- Peter Geoghegan