Re: EXPLAIN VERBOSE with parallel Aggregate - Mailing list pgsql-hackers

From David Rowley
Subject Re: EXPLAIN VERBOSE with parallel Aggregate
Date
Msg-id CAKJS1f-Wzf1UCmC1yS6_amUHsQPOK9+CvE856BQ6-a2==T0zdA@mail.gmail.com
Whole thread Raw
In response to Re: EXPLAIN VERBOSE with parallel Aggregate  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
On 28 April 2016 at 04:08, Robert Haas <robertmhaas@gmail.com> wrote:
> On Tue, Apr 26, 2016 at 11:38 PM, David Rowley
>> The patch looks good. The only other thing I thought about was perhaps
>> it would be a good idea to re-add the sanity checks in execQual.c.
>> Patch for that is attached.
>>
>> I removed the aggoutputtype check, as I only bothered adding that in
>> the first place because I lost the aggpartial field in some previous
>> revision of the parallel aggregate developments. I'd say the
>> aggpartial check makes it surplus to requirements.
>
> OK, committed.

Great. Thank you.

-- David Rowley                   http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training & Services



pgsql-hackers by date:

Previous
From: Thomas Munro
Date:
Subject: Re: Shared memory and processes
Next
From: Michael Paquier
Date:
Subject: Re: Shared memory and processes