| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Nathan Bossart <nathandbossart(at)gmail(dot)com>, Paul Ramsey <pramsey(at)cleverelephant(dot)ca>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Pg18 Recursive Crash |
| Date: | 2024-12-18 01:52:05 |
| Message-ID: | CAApHDvqAqKccqUtdraNtWcKpFDH6Ngzdtq8hSMmCj=+32X6gHw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, 18 Dec 2024 at 14:02, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> So I tried adapting my patch to not make a copy of the input slot,
> and it didn't work: I was still getting assertion failures about
> the slot not being a MinimalTupleSlot as expected. On investigation
> it appears your patch did not fully adjust BuildTupleHashTableExt
> for variable input-slot type. You need the attached as well.
Do you have a test case in master that triggers a problem here? Your
patch adjusts code that existed prior to d96d1d515, so I'm confused as
to why your patch is needed now when it wasn't before.
If you're only triggering an issue after patching with your setops
patch, are your changes maybe using FindTupleHashEntry() with an
eqcomp that isn't compatible with the 'slot' parameter you're passing
to that function?
David
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Kohei Harikae (Fujitsu) | 2024-12-18 02:05:20 | RE: Windows meson build |
| Previous Message | wenhui qiu | 2024-12-18 01:50:51 | Re: Add 64-bit XIDs into PostgreSQL 15 |