Re: [HACKERS] proposal - Default namespaces for XPath expressions(PostgreSQL 11) - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: [HACKERS] proposal - Default namespaces for XPath expressions(PostgreSQL 11)
Date
Msg-id CAFj8pRCHRhG6AWnU5VKHsvA_L3DjQOnUvT=eRdSPz0qfA6ig4Q@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] proposal - Default namespaces for XPath expressions(PostgreSQL 11)  (Andres Freund <andres@anarazel.de>)
Responses Re: [HACKERS] proposal - Default namespaces for XPath expressions(PostgreSQL 11)  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers


čt 31. 1. 2019 v 14:44 odesílatel Andres Freund <andres@anarazel.de> napsal:
Hi,

On 2018-09-18 11:33:38 -0400, Tom Lane wrote:
> Pavel Stehule <pavel.stehule@gmail.com> writes:
> > [ xml-xpath-default-ns-7.patch ]
>
> At Andrew's prompting, I took a look over this patch.  I don't know much
> of anything about XML, so I have no idea as to standards compliance here,
> but I do have some comments:
>
> * I'm fairly uncomfortable with the idea that we're going to maintain
> our own XPath parser.  That seems like a recipe for lots of future
> work ... and the code is far too underdocumented for anyone to actually
> maintain it.  (Case in point: _transformXPath's arguments are not
> documented at all, and in fact there's hardly a word about what it's
> even supposed to *do*.)

We were looking at this patch at the pgday developer meeting: Our
impression is that this patch should be rejected. If really desired, the
best approach seems to be actually implement this in libxml, and then go
from there.

Unfortunately,  the development of libxml2 is frozen.

I have to accept it.

Regards

Pavel



Greetings,

Andres Freund

pgsql-hackers by date:

Previous
From: John Naylor
Date:
Subject: Re: WIP: Avoid creation of the free space map for small tables
Next
From: Daniel Gustafsson
Date:
Subject: Re: Allow auto_explain to log to NOTICE