Re: Converting contrib SQL functions to new style

On Thu, Nov 07, 2024 at 10:06:37AM +0900, Michael Paquier wrote:
> Good point. Checking all these contrib updates one-by-one is an ant's
> work, but I'll see if I can get at least some of them done on HEAD.

I've begun looking at that a bit, and there are a couple of things
that we could do better with xml2 in 0005 at least in the context of
this patch: xpath_nodeset() and xpath_list() don't have any test
coverage. That's not an issue directly related to this patch, but
perhaps we should add something for the functions that we are
manipulating after this upgrade path at least? That's one way to
automatically make sure that these changes work the same way as the
original.

The same argument comes up with lo_oid() in 0006.

0004 for pg_freespace is fine regarding that for example as we have
calls of pg_freespace(regclass) in its sql/. I've applied it to begin
with something.

Tomas Vondra has posted a patch for a bug fix with pageinspect, so
this would create some conflicts noise for him if 0003 was applied
today, so let's wait a bit:
https://www.postgresql.org/message-id/3385a58f-5484-49d0-b790-9a198a0bf236@vondra.me

0001 and 0002 are much larger than the 4 others, and I'm lacking the
steam to check them in more details today.
--
Michael