Re: [HACKERS] pg_dump emits ALTER TABLE ONLY partitioned_table - Mailing list pgsql-hackers

From Amit Langote
Subject Re: [HACKERS] pg_dump emits ALTER TABLE ONLY partitioned_table
Date
Msg-id 3e2aa1e9-e621-9ad5-b24d-e24dbc9650ba@lab.ntt.co.jp
Whole thread Raw
In response to Re: [HACKERS] pg_dump emits ALTER TABLE ONLY partitioned_table  (Stephen Frost <sfrost@snowman.net>)
Responses Re: [HACKERS] pg_dump emits ALTER TABLE ONLY partitioned_table
List pgsql-hackers
Hi Stephen,

On 2017/05/06 12:28, Stephen Frost wrote:
> Noah,
> 
> On Fri, May 5, 2017 at 23:19 Noah Misch <noah@leadboat.com> wrote:
> 
>> On Thu, May 04, 2017 at 05:47:02PM -0400, Stephen Frost wrote:
>>> * Amit Langote (amitlangote09@gmail.com) wrote:
>>>> On Wed, May 3, 2017 at 12:05 PM, Stephen Frost <sfrost@snowman.net>
>> wrote:
>>>>> Assuming this looks good to you, I'll push it tomorrow, possibly with
>>>>> other minor adjustments and perhaps a few more tests.
>>>>
>>>> Your latest patch looks good to me.
>>>
>>> Found a few more issues (like pg_dump not working against older versions
>>> of PG, because the queries for older versions hadn't been adjusted) and
>>> did a bit more tidying.
>>>
>>> I'll push this in a couple of hours.
>>
>> This PostgreSQL 10 open item is past due for your status update.  Kindly
>> send
>> a status update within 24 hours, and include a date for your subsequent
>> status
>> update.  Refer to the policy on open item ownership:
>>
>> https://www.postgresql.org/message-id/20170404140717.GA2675809%40tornado.leadboat.com
>>
>> I see you did commit a related patch.  Is this item done?
> 
> 
> Yes, I believe it to be. I anticipate reviewing the code in this area more
> in the coming weeks, but this specific issue can be marked as resolved.

Thanks for committing the patch after improving it quite a bit, and sorry
that I couldn't reply promptly during the last week due to vacation.

Regards,
Amit




pgsql-hackers by date:

Previous
From: Thomas Munro
Date:
Subject: Re: [HACKERS] modeling parallel contention (was: Parallel Append implementation)
Next
From: Michael Paquier
Date:
Subject: Re: [HACKERS] Concurrent ALTER SEQUENCE RESTART Regression