On Mon, Oct 14, 2024 at 3:05 AM Peter Eisentraut <pe...@eisentraut.org> wrote: > > On 14.10.24 08:12, Bertrand Drouvot wrote: > >> It seems that src/backend/utils/adt/float.c file still has functions > >> that can use [de]construct_array_builtin(), for example > >> float8_combine(). I think it's an oversight of d746021de1. > > Thanks for looking at it. > > > > Good catch, please find attached v2 taking care of those. I looked at the > > remaining [de]construct_array() usages and that looks ok to me. > > This looks good to me. I can't think of a reason why it was not > included in the original patch.
Thank you for reviewing the patch. Pushed (v2 patch). Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com