| From: | "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com> |
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Cc: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, vignesh C <vignesh21(at)gmail(dot)com> |
| Subject: | RE: Added missing invalidations for all tables publication |
| Date: | 2021-09-07 09:14:31 |
| Message-ID: | OS0PR01MB5716BC64682DEE780536739394D39@OS0PR01MB5716.jpnprd01.prod.outlook.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
From Mon, Sep 6, 2021 1:56 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> On Tue, Aug 31, 2021 at 8:54 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
> > Thanks for the comments, the attached v3 patch has the changes for the
> > same.
> >
>
> I think this bug should be fixed in back branches (till v10). OTOH, as this is not
> reported by any user and we have found it during code review so it seems
> either users don't have an exact use case or they haven't noticed this yet. What
> do you people think about back-patching?
Personally, I think it's ok to back-patch.
> Attached, please find a slightly updated patch with minor changes. I have
> slightly changed the test to make it more meaningful.
The patch looks good to me.
Best regards,
Hou zj
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2021-09-07 09:22:24 | Re: Remove Value node struct |
| Previous Message | Seino Yuki | 2021-09-07 09:11:14 | Re: Add statistics refresh materialized view |