On Tue, Jul 4, 2017 at 9:51 AM, Amit Langote
> Attached updated patches.
There's an extra "we" in
+ * Note that attachRel's OID is in this list. If it's partitioned, we
+ * we don't need to schedule it to be scanned (would be a noop anyway
And some portions of the comment before find_all_inheritors() in
ATExecAttachPartition() look duplicated in portions of the code that
check constraints on the table being attached and each of its leaf
Other than that the patches look good to me.
The Postgres Database Company
Sent via pgsql-hackers mailing list (email@example.com)
To make changes to your subscription: