Re: satisfies_hash_partition crash

Ewan Young <kdbase.hack@gmail.com>

From: Ewan Young <kdbase.hack@gmail.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Tender Wang <tndrwang@gmail.com>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2026-07-03T02:07:46Z
Lists: pgsql-hackers
On Fri, Jul 3, 2026 at 1:54 AM Robert Haas <robertmhaas@gmail.com> wrote:
>
> On Thu, Jul 2, 2026 at 2:17 AM Tender Wang <tndrwang@gmail.com> wrote:
> > Personally,  I prefer a local bool variable near the top and reuse it.
>
> I prefer to more thoroughly confine this fix to the variadic code
> path, since I don't think we really expect that to be exercised in
> practice. So here's my proposal.

v2 looks good to me — I tested it and confirmed that it works. Thanks
for picking this up.

>
> --
> Robert Haas
> EDB: http://www.enterprisedb.com

-- 
Regards,
Ewan Young



Commits

  1. Fix multiple problems with satisfies_hash_partition.