Re: Shouldn't construct_array_builtin and deconstruct_array_builtin agree on types? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Shouldn't construct_array_builtin and deconstruct_array_builtin agree on types?
Date
Msg-id 1389829.1686616005@sss.pgh.pa.us
Whole thread Raw
In response to Re: Shouldn't construct_array_builtin and deconstruct_array_builtin agree on types?  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
Michael Paquier <michael@paquier.xyz> writes:
> On Mon, Jun 12, 2023 at 11:06:18PM +0200, Peter Eisentraut wrote:
>> They only support the types that they were actually being used with.  If you
>> need another type, feel free to add it.

> FWIW, I agree with Tomas that this is an oversight that should be
> fixed in v16, saving from the need to have a copy of
> deconstruct_array_builtin() in extensions.

We don't want to bloat these functions indefinitely, so I understand
Peter's approach of only adding the cases actually being used.
At the same time, it's reasonable to take some thought for extensions
that might want slightly more functionality than the core code
happens to need at any given moment.

The idea of making both functions support the same set of types
does seem like a reasonable compromise.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: query_id, pg_stat_activity, extended query protocol
Next
From: "Fujii.Yuki@df.MitsubishiElectric.co.jp"
Date:
Subject: RE: Partial aggregates pushdown