From: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
---|---|
To: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
Cc: | Murat Tuncer <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: BUG #14759: insert into foreign data partitions fail |
Date: | 2017-08-01 01:10:07 |
Message-ID: | CAKFQuwZNxLfpsUPMs82m+t-Z3Oa45jVREjLqima5GOx3DK7NrQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
On Mon, Jul 31, 2017 at 5:42 PM, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp
> wrote:
>
> On a second thought though, I think we should list the foreign table
> partitions' limitations in only one place, that is, the CREATE FOREIGN
> TABLE reference page. Listing them under 5.10.2.3. seems a bit off to me,
> because other limitations listed there are those of the new partitioned
> table objects, such as lack of global index constraints, etc. Lack of
> tuple-routing to foreign partitions does not seem to me of the similar
> nature. Also, the same text is no longer repeated in 3 different places.
>
> Thoughts on the updated patch?
>
Overall, works for me.
grammar (add a couple of commas for flow) and style (dropping the first
"the")
current: "(both the user-defined constraints such as <literal>CHECK</> or
<literal>NOT NULL</> clauses and the partition constraint)"
proposed: "(both user-defined constraints, such as <literal>CHECK</> or
<literal>NOT NULL</> clauses, and the partition constraint)"
Thanks!
David J.
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-08-01 01:14:07 | Re: BUG #14758: Segfault with logical replication on a function index |
Previous Message | Amit Langote | 2017-08-01 00:42:13 | Re: BUG #14759: insert into foreign data partitions fail |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-08-01 01:11:39 | Re: Parallel Hash take II |
Previous Message | Peter Eisentraut | 2017-08-01 00:54:28 | Re: PL_stashcache, or, what's our minimum Perl version? |