MIME-Version: 1.0
References: 
 <CAHewXNmcXVFdB9_WwA8Ez0P+m_TQy_KzYk5Ri5dvg+fuwjD_yw@mail.gmail.com>
 <CAApHDvrSSeLNWzQWWcLfZKWwHd=XocVs3abmTmvaSD_sx-dkNQ@mail.gmail.com>
 <CAApHDvrRR1VOk4i4CpNWeL48veFshfRAvDTuWxsiUhUqo0akwQ@mail.gmail.com>
 <CAApHDvreF-UiqBaHtRTQWQ6z1X9snstJW+dfb2DU5GOb-uPEBg@mail.gmail.com>
 <CAHewXN=Zzy7kPf3kSkxRzMY9tSZFOt8JRhKW_M_FzLY3px-esQ@mail.gmail.com>
In-Reply-To: 
 <CAHewXN=Zzy7kPf3kSkxRzMY9tSZFOt8JRhKW_M_FzLY3px-esQ@mail.gmail.com>
From: David Rowley <dgrowleyml@gmail.com>
Date: Mon, 30 Mar 2026 16:20:33 +1300
Message-ID: 
 <CAApHDvo=qkcxc-Lx83XOxkC9xNfp1tkCZ6Z5rQL5L1vXv=+a7Q@mail.gmail.com>
Subject: Re: Fix "could not find memoization table entry"
To: Tender Wang <tndrwang@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAApHDvo%3Dqkcxc-Lx83XOxkC9xNfp1tkCZ6Z5rQL5L1vXv%3D%2Ba7Q%40mail.gmail.com>
Precedence: bulk

On Wed, 25 Mar 2026 at 22:00, Tender Wang <tndrwang@gmail.com> wrote:
>
> David Rowley <dgrowleyml@gmail.com> =E4=BA=8E2026=E5=B9=B43=E6=9C=8825=E6=
=97=A5=E5=91=A8=E4=B8=89 10:09=E5=86=99=E9=81=93=EF=BC=9A
> >
> > On Wed, 25 Mar 2026 at 13:31, David Rowley <dgrowleyml@gmail.com> wrote=
:
> > I was experimenting with a less risky fix by having the datum_image
> > functions force the sign-extended representation of the Datum before
> > hashing or comparing.
> >
> > Attached is a quick PoC of what that would look like. It does fix the
> > reported problem. But it is a hack and doesn't fix the root cause of
> > the issue.
> >
> > Despite the hackiness, I feel this might be better than the
> > whack-a-mole approach of just fixing incorrect usages of the
> > PG_RETURN_* macros as and when we find them.
>
> No objection from me.
> It seems no users have complained about hash_numberic(), and except
> for this reported issue, no internal errors have been reported due to
> hash_numberic().

I cleaned that method up and pushed it.

Thanks for the report.

David