Re: Add proper planner support for ORDER BY / DISTINCT aggregates

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Greg Stark <stark(at)mit(dot)edu>
Cc: Ronan Dunklau <ronan(dot)dunklau(at)aiven(dot)io>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Ranier Vilela <ranier(dot)vf(at)gmail(dot)com>
Subject: Re: Add proper planner support for ORDER BY / DISTINCT aggregates
Date: 2022-04-07 04:09:41
Message-ID: CAApHDvqN98xhm3=MjtbObYmWu=Z2EZc=hG1FYzKALPeJbNLKGw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, 31 Mar 2022 at 06:36, Greg Stark <stark(at)mit(dot)edu> wrote:
>
> This patch is now failing in the sqljson regression test. It looks
> like it's just the ordering of the elements in json_arrayagg() calls
> which may actually be a faulty test that needs more ORDER BY clauses
> rather than any issues with the code. Nonetheless it's something that
> needs to be addressed before this patch could be applied.
>
> Given it's gotten some feedback from Ronan and this regression test
> failure I'll move it to Waiting on Author but we're near the end of
> the CF and it'll probably be moved forward soon.

Thanks for mentioning this and for keeping tabs on it.

This patch in general is more than there's realistic time for in this
CF. I'd very much like to get the DISTINCT part working too. Not just
the ORDER BY. I've pushed this one out to July's CF now.

David

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2022-04-07 05:34:16 Re: Logical replication timeout problem
Previous Message Peter Smith 2022-04-07 04:09:23 Re: Handle infinite recursion in logical replication setup