Re: Add ExprState hashing for GROUP BY and hashed SubPlans

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Andrei Lepikhov <lepihov(at)gmail(dot)com>
Cc: PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Add ExprState hashing for GROUP BY and hashed SubPlans
Date: 2024-10-31 01:16:07
Message-ID: CAApHDvpnF=16BYa74_Mn9RX=1Ctt6JgaHmcfhd4MdHJWUwVuyg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, 29 Oct 2024 at 22:47, David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
> I've attached an updated patch with a few other fixes. Whilr checking
> this tonight, noticed that master does not use
> SubPlanState.tab_eq_funcs for anything. I resisted removing that in
> this patch. Perhaps a follow-on patch can remove that. I suspect it's
> not been used for a long time now, but I didn't do the archaeology
> work to find out.

3974bc319 removed the SubPlanState.tab_eq_funcs field, so here's a
rebased patch.

David

Attachment Content-Type Size
v3-0001-Use-ExprStates-for-hashing-in-GROUP-BY-and-SubPla.patch application/octet-stream 19.1 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Yushi Ogiwara 2024-10-31 01:22:11 Making error message more user-friendly with spaces in a URI
Previous Message Andy Fan 2024-10-31 01:11:14 Re: Avoid detoast overhead when possible