From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrey Borodin <x4mmm(at)yandex-team(dot)ru> |
Cc: | Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com>, Michel Pelletier <pelletier(dot)michel(at)gmail(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Using Expanded Objects other than Arrays from plpgsql |
Date: | 2025-02-03 18:18:10 |
Message-ID: | 416464.1738606690@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general pgsql-hackers |
Andrey Borodin <x4mmm(at)yandex-team(dot)ru> writes:
>> On 3 Feb 2025, at 22:36, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> I'm not wedded to that name; do you have a better idea?
> I'd propose something like attached. But feel free to ignore my suggestion: I do not understand context of these structure members.
Hmm, you're suggesting naming those field members after PL/pgSQL's
specific use of them. But the intent was that they are generic
workspace for anything that provides a EEOP_PARAM_CALLBACK
callback --- that is, the "param" in the field name refers to the
fact that this is an expression step for some kind of Param, and
not to what PL/pgSQL happens to do with the field.
Admittedly this is all moot unless some other extension starts
using EEOP_PARAM_CALLBACK, and I didn't find any evidence of that
using Debian Code Search. But I don't want to think of
EEOP_PARAM_CALLBACK as being specifically tied to PL/pgSQL.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2025-02-03 18:48:56 | Re: Using Expanded Objects other than Arrays from plpgsql |
Previous Message | Andrey Borodin | 2025-02-03 17:57:27 | Re: Using Expanded Objects other than Arrays from plpgsql |
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2025-02-03 18:41:03 | Re: Skip collecting decoded changes of already-aborted transactions |
Previous Message | Tom Lane | 2025-02-03 18:07:59 | Re: Better title output for psql \dt \di etc. commands |