Re: Error when using array_agg with filter where clause in pg16 and pg17

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Kaimeh <kkaimeh(at)gmail(dot)com>, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: Error when using array_agg with filter where clause in pg16 and pg17
Date: 2025-04-09 00:54:50
Message-ID: CAApHDvoAToLkMqJVMLnML61u2itZRfc5At5eMd_VOxY_jer0Eg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Wed, 9 Apr 2025 at 12:52, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> David Rowley <dgrowleyml(at)gmail(dot)com> writes:
> > I suspect we should just leave this for v18 and maybe come back and
> > improve for v19.
>
> I think not doing anything is unacceptable: even though it took awhile
> to notice, presorted_agg flat out breaks some queries that worked
> before. That trumps any worries about "maybe the plan will be worse",
> and I don't even think it's a close decision. So my inclination is
> to do the simplest possible thing in v16-v18, and that seems to be
> to disable presorted_agg if there's a FILTER. Then we can look
> into better ideas at leisure for v19.

Misunderstanding. I meant do nothing about the costing issue. I still
think we should fix the bug, of course. The POC patch I posted is for
that part.

David

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message David Rowley 2025-04-09 01:50:48 Re: Error when using array_agg with filter where clause in pg16 and pg17
Previous Message Tom Lane 2025-04-09 00:52:26 Re: Error when using array_agg with filter where clause in pg16 and pg17