Re: BUG #16631: postgres_fdw tries to insert into generated columns

From: Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com>
To: Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, dacherny(at)gmail(dot)com, PostgreSQL mailing lists <pgsql-bugs(at)lists(dot)postgresql(dot)org>
Subject: Re: BUG #16631: postgres_fdw tries to insert into generated columns
Date: 2021-08-04 08:30:44
Message-ID: CAPmGK15Ze73T+U=P2-gyGXz8tbGHN371B=5c4WVXiPqy0u7VOA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Thu, Jul 15, 2021 at 6:15 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> On Tue, Jul 13, 2021 at 5:36 AM Peter Eisentraut
> <peter(dot)eisentraut(at)enterprisedb(dot)com> wrote:

> > The follow-up proposal was to let the FDW implementation decide. I
> > don't think that is always the right thing either. What if I want the
> > foreign-table node to do the computations, because it has access to
> > special parameters or something.
>
> The follow-up proposal was mainly for consistency with the handling of
> constraints on foreign tables; they are not enforced on the local PG
> server at all, as noted in the above documentation.
>
> > There might be multiple possibilities of what is appropriate here. But
> > that's not something we can just change around as part of a bug fix.
>
> I agree on this part. The changes I made to the core side would go
> beyond a fix for the reported issue. I'll remove it in the next
> version of the patch.

I updated the patch as such. Attached is an update patch
(postgres_fdw-generated-columns-PG14-HEAD.patch), which would apply to
HEAD and v14. I'm also attaching a patch for v12 and v13
(postgres_fdw-generated-columns-PG12-PG13.patch). I'll apply these if
there are no objections.

Best regards,
Etsuro Fujita

Attachment Content-Type Size
postgres_fdw-generated-columns-PG14-HEAD.patch application/octet-stream 21.5 KB
postgres_fdw-generated-columns-PG12-PG13.patch application/octet-stream 17.6 KB

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Ján Pecsők 2021-08-04 15:15:42 CAST from numeric(18,3) to numeric doesnt work, posgresql 13.3
Previous Message Kyotaro Horiguchi 2021-08-04 02:05:00 Re: psql doesn't show tables duplicated in multiple schemas