Re: [HACKERS] BUG #14759: insert into foreign data partitions fail

From: Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>
To: Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
Cc: mtuncer(at)gmail(dot)com, "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] BUG #14759: insert into foreign data partitions fail
Date: 2017-07-31 00:58:04
Message-ID: bfbfb80b-6a85-5fbb-674f-cd05aa54d60a@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Thank you for weighing in and reviewing the patch.

On 2017/07/28 20:55, Etsuro Fujita wrote:
> On 2017/07/26 15:29, Amit Langote wrote:
>> On 2017/07/25 9:43, David G. Johnston wrote:
>>> On Mon, Jul 24, 2017 at 5:19 PM, Amit Langote wrote:
>>> I'm curious what the other limitations are...
>
> I think COPY has the same limitation as INSERT.

Yes. I updated the patch to mention that as well.

>> When I first wrote that documentation line (I am assuming you're asking
>> about "although these have some limitations that normal tables do not"), I
>> was thinking about the fact that the core system does not enforce
>> (locally) any constraints defined on foreign tables. Since we allow
>> inserting data into partitions directly, it is imperative that we enforce
>> the "partition constraint" along with the traditional constraints such as
>> NOT NULL and CHECK constraints, which we can do for local table partitions
>> but not for foreign table ones.
>>
>> Anyway, attached patch documents all these limitations about foreign table
>> partitions more prominently.
>
> Typo: s/the they is/they are/

Fixed in the attached.

Thanks,
Amit

Attachment Content-Type Size
0001-Clarify-that-partition-constraint-is-not-enforced-on-v2.patch text/plain 3.8 KB

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Rick Otten 2017-07-31 01:05:50 Re: signal 11 segfaults with parallel workers
Previous Message Masahiko Sawada 2017-07-31 00:40:34 Re: BUG #14758: Segfault with logical replication on a function index

Browse pgsql-hackers by date

  From Date Subject
Next Message Steve Singer 2017-07-31 01:34:59 10 beta docs: different replication solutions
Previous Message Masahiko Sawada 2017-07-31 00:42:45 Re: Refreshing subscription relation state inside a transaction block