| 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: | Whole Thread | Raw Message | 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 |
| 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 |