Re: [HACKERS] [Proposal] Make the optimiser aware of partitions ordering

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Julien Rouhaud <rjuju123(at)gmail(dot)com>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] [Proposal] Make the optimiser aware of partitions ordering
Date: 2017-11-29 02:14:05
Message-ID: CAB7nPqQuvRZtz0rpPR0DKEb=mKjioTPFZdpPiqW1O16TsDZ7og@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Sep 27, 2017 at 2:59 AM, Julien Rouhaud <rjuju123(at)gmail(dot)com> wrote:
> On Tue, Sep 26, 2017 at 2:56 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>> On Sat, Sep 23, 2017 at 6:29 AM, Julien Rouhaud <rjuju123(at)gmail(dot)com> wrote:
>>> That's true, but numCols, sortColdIdx etc are also used to display the
>>> sort key in an explain. If an append can return sorted data, it
>>> should also display the sort information, so I think these fields are
>>> still required in an Append node.
>>
>> I don't think so. An index scan doesn't output that information, nor
>> does a nested loop which inherits a sort order from its outer path. I
>> think the rule is that a plan node which takes steps to get the data
>> into a certain order might want to output something about that, but a
>> plan node which somehow gets that ordering without taking any explicit
>> action doesn't print anything.
>
> Oh, ok that indeed makes sense. As I said I'll remove all the useless
> noise and send an updated patch. Thanks again.

Waiting for a patch update for two months now, so marked as returned
with feedback.
--
Michael

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2017-11-29 02:16:49 Re: [HACKERS] WIP: Covering + unique indexes.
Previous Message Craig Ringer 2017-11-29 02:13:32 Re: [PATCH] Atomic pgrename on Windows