Michael Paquier <[email protected]> writes:
> On Wed, Sep 02, 2026 at 07:56:32PM -0400, Andres Freund wrote:
>> FWIW, I think we should seriously consider making [almost] all the transition
>> states internal. Having to support calling these functions in non-aggregate
>> contexts adds complexity without any actual gain. We shouldn't need to check
>> whether we are in an AggContext, whether the argument is toasted, whether
>> there are NULL elements in the array, compute offsets into the array, etc.
> +1.
Seems reasonable. We could still have the actual representation be
int8[] or whatever if it saves code to do so, but we could declare
it internal at SQL level to remove these edge cases.
regards, tom lane
