Re: Fix xpath() to return namespace definitions

From: Ali Akbar <the(dot)apaan(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix xpath() to return namespace definitions
Date: 2014-06-04 22:35:14
Message-ID: CACQjQLr=vxvrU2JqYBg7v7r=hjMOKn9_n-z-ZWRKVOzHa47R-A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2014-06-05 2:37 GMT+07:00 Robert Haas <robertmhaas(at)gmail(dot)com>:

> Please add your patch here so we don't forget about it:
>
> https://commitfest.postgresql.org/action/commitfest_view/open
>

Added: https://commitfest.postgresql.org/action/patch_view?id=1461

> Please see also:
>
> https://wiki.postgresql.org/wiki/Submitting_a_Patch
>

Thanks, i've read it.
As for the suggestion there: "Start out by reviewing a patch or responding
to email on the lists. Even if it is unrelated to what you're doing", i'll
look around other posts in this list.

For back versions, i think because this patch changes xpath() behavior, we
will only apply this to future versions. The old behavior is wrong
(according to XPath standard) for not including namespaces, but maybe there
are some application that depends on the old behavior.

Thanks!
--
Ali Akbar

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2014-06-04 22:57:31 Re: pg_control is missing a field for LOBLKSIZE
Previous Message Tom Lane 2014-06-04 22:00:45 Re: Sigh, we need an initdb