Re: Compatible defaults for LEAD/LAG - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: Compatible defaults for LEAD/LAG
Date
Msg-id CAFj8pRC35Zt5v9EqLxoq1bbLjMTWBTGys2zpd5E+G0s7bG_M3w@mail.gmail.com
Whole thread Raw
In response to Re: Compatible defaults for LEAD/LAG  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers


st 4. 11. 2020 v 22:12 odesílatel Tom Lane <tgl@sss.pgh.pa.us> napsal:
Pavel Stehule <pavel.stehule@gmail.com> writes:
> út 22. 9. 2020 v 2:33 odesílatel Tom Lane <tgl@sss.pgh.pa.us> napsal:
>> Anyway, attached find
>> 0001 - updates Vik's original patch to HEAD and tweaks the
>> grammar in the docs a bit.
>> 0002 - add-on patch to convert array_append, array_prepend,
>> array_cat, array_position, array_positions, array_remove,
>> array_replace, and width_bucket to use anycompatiblearray.
>> I think 0001 is committable, but 0002 is just WIP since
>> I didn't touch the docs.  I'm slightly discouraged about
>> whether 0002 is worth proceeding with.  Any thoughts?

> I think so 0002 has sense - more than doc I miss related regress tests, but
> it is partially covered by anycompatible tests

I didn't see any need for particularly exhaustive testing, but
I did add one new test for an operator and one for a function.
Pushed with that and the necessary docs work.

ok, Thank you

Pavel
 

                        regards, tom lane

pgsql-hackers by date:

Previous
From: David Rowley
Date:
Subject: Re: Use of "long" in incremental sort code
Next
From: Tomas Vondra
Date:
Subject: Re: Use of "long" in incremental sort code