I think this is ready for committer.

On Thu, Feb 23, 2017 at 12:02 PM, Amit Langote
<langote_amit...@lab.ntt.co.jp> wrote:
> On 2017/02/22 21:24, Ashutosh Bapat wrote:
>> On Wed, Feb 22, 2017 at 11:11 AM, Amit Langote wrote:
>>> +       /*
>>> +        * Unlike inheritance children, partition tables are expected to be 
>>> dropped
>>> +        * when the parent partitioned table gets dropped.
>>> +        */
>>>
>>> Hmm.  Partitions *are* inheritance children, so we perhaps don't need the
>>> part before the comma.  Also, adding "automatically" somewhere in there
>>> would be nice.
>>>
>>> Or, one could just write: /* add an auto dependency for partitions */
>>
>> I changed it in the attached patch to
>> +    /*
>> +     * Partition tables are expected to be dropped when the parent 
>> partitioned
>> +     * table gets dropped.
>> +     */
>
> OK, thanks.
>
> Thanks,
> Amit
>
>



-- 
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to