On Wed, Jun 14, 2017 at 10:21 AM, Ashutosh Bapat
> On Tue, Jun 13, 2017 at 7:14 PM, Rahila Syed <rahilasye...@gmail.com> wrote:
>> I reviewed and tested
>> It applies cleanly on master and make check passes.
>> Following are few comments:
>>> * Drop the dependency created by StoreCatalogInheritance1 (CREATE TABLE
>>> * INHERITS/ALTER TABLE INHERIT -- refclassid will be RelationRelationId)
>>> * heap_create_with_catalog (CREATE TABLE OF/ALTER TABLE OF -- refclassid
>>> * be TypeRelationId). There's no convenient way to do this, so go
>>> * through pg_depend.
>>>drop_parent_dependency(Oid relid, Oid refclassid, Oid refobjid,
>> DependencyType deptype)
>> This is not directly related to this patch but still would like to mention.
>> drop_parent_dependency() is being used to drop the dependency created
>> by CREATE TABLE PARTITION OF command also, so probably the comment
>> needs to be modified to reflect that.
> The comment is fine for dependency created by CREATE TABLE PARTITION
> OF since that too goes through StoreCatalogInheritance1(). But it's
> not correct for CREATE TABLE ... OF <composite type> since that does
> not go through StoreCatalogInheritance1().
I missed "or" part of the prologue while writing this. Now that I have
read it carefully, the prologue looks good to me.
The Postgres Database Company
Sent via pgsql-hackers mailing list (firstname.lastname@example.org)
To make changes to your subscription: