From: | Vojtěch Beneš <vojtech(dot)benes(at)centrum(dot)cz> |
---|---|
To: | <dgrowleyml(at)gmail(dot)com>, <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | <pgsql-bugs(at)lists(dot)postgresql(dot)org> |
Subject: | Re: BUG #18264: Table has type text, but query expects integer.attribute 1 of type record has wrong type |
Date: | 2024-01-05 17:39:32 |
Message-ID: | 20240105183932.2D7B2BB4@centrum.cz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
I am impressed with how quickly you addressed the issue. I tested queries myself on latest builds and issue is resolved. Also all errors we were getting in more complex queries are gone.
Thank you for your work.
Vojtěch
______________________________________________________________
> Od: "David Rowley" <dgrowleyml(at)gmail(dot)com>
> Komu: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> Datum: 04.01.2024 08:43
> Předmět: Re: BUG #18264: Table has type text, but query expects integer.attribute 1 of type record has wrong type
>
> CC: <pgsql-bugs(at)lists(dot)postgresql(dot)org>
On Sun, 31 Dec 2023 at 16:24, David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
> It looks like the problem is in ExecEvalPreOrderedDistinctMulti(). It
> overwrites the tmpcontext's slots but fails to properly restore the
> originals again.
I pushed a slightly revised version of this. I ended up adding a bool
isdistinct local variable so I didn't have to reset the slots in two
places. It kinda makes it easier to see what the true and false
means, so that might have been a good change regardless of the bug.
Thanks for reporting this Vojtěch and for the reproducer script.
David
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2024-01-05 17:56:49 | Re: BUG #17946: LC_MONETARY & DO LANGUAGE plperl - BUG |
Previous Message | Wladimir | 2024-01-05 15:52:53 | Re: BUG #18271: Re: Postgres policy exists bug |