On Tue, Aug 31, 2021 at 2:00 PM Kyotaro Horiguchi
<horikyota.ntt@gmail.com> wrote:
>
> At Tue, 31 Aug 2021 08:31:05 +0530, vignesh C <vignesh21@gmail.com> wrote in
> > On Tue, Aug 31, 2021 at 7:40 AM houzj.fnst@fujitsu.com
> > <houzj.fnst@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.
>
Thanks for the comments, the attached v3 patch has the changes for the same.
Regards,
Vignesh