Re: Added missing invalidations for all tables publication

On Tue, Aug 31, 2021 at 2:00 PM Kyotaro Horiguchi
<horikyota(dot)ntt(at)gmail(dot)com> wrote:
>
> At Tue, 31 Aug 2021 08:31:05 +0530, vignesh C <vignesh21(at)gmail(dot)com> wrote in
> > On Tue, Aug 31, 2021 at 7:40 AM houzj(dot)fnst(at)fujitsu(dot)com
> > <houzj(dot)fnst(at)fujitsu(dot)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