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
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 |