At Tue, 31 Aug 2021 08:31:05 +0530, vignesh C <vignes...@gmail.com> wrote in > On Tue, Aug 31, 2021 at 7:40 AM houzj.f...@fujitsu.com > <houzj.f...@fujitsu.com> wrote: > Thanks for the comment, I have slightly modified the test case which > will fail without the patch. Attached v2 patch which has the changes > for the same.
The test works fine. The code looks fine for me except one minor cosmetic flaw. + if (!HeapTupleIsValid(tup)) + elog(ERROR, "cache lookup failed for publication %u", + pubid); The last two lines don't need to be separated. ((Almost) All other instance of the same error is written that way. regards. -- Kyotaro Horiguchi NTT Open Source Software Center
