On Fri, Jul 5, 2019 at 3:03 PM Jamison, Kirk <k.jami...@jp.fujitsu.com> wrote: > I updated the patch which is similar to V3 of the patch, > but addressing my problem in (5) in the previous email regarding > FreeSpaceMapVacuumRange. > It seems to pass the regression test now. Kindly check for validation.
Hi Kirk, FYI there are a couple of compiler errors reported: Windows compiler: contrib/pg_visibility/pg_visibility.c(400): error C2143: syntax error : missing ')' before '{' [C:\projects\postgresql\pg_visibility.vcxproj] GCC: storage.c: In function ‘RelationTruncate’: storage.c:238:14: error: variable ‘newnblocks’ set but not used [-Werror=unused-but-set-variable] BlockNumber newnblocks = InvalidBlockNumber; ^ storage.c:237:14: error: variable ‘new_nfsmblocks’ set but not used [-Werror=unused-but-set-variable] BlockNumber new_nfsmblocks = InvalidBlockNumber; ^ storage.c: In function ‘smgr_redo’: storage.c:634:15: error: variable ‘newnblocks’ set but not used [-Werror=unused-but-set-variable] BlockNumber newnblocks = InvalidBlockNumber; ^ storage.c:633:15: error: variable ‘new_nfsmblocks’ set but not used [-Werror=unused-but-set-variable] BlockNumber new_nfsmblocks = InvalidBlockNumber; ^ -- Thomas Munro https://enterprisedb.com