Mike Fowler <mike(at)mlfowler(dot)com> writes:
> On 06/08/10 20:55, Peter Eisentraut wrote:
>> On fre, 2010-08-06 at 09:04 +0100, Mike Fowler wrote:
>>> If the patch is to be committed, does it make sense for me to refine
>>> it such that it uses the new xpath internal function you extracted in
>>> the xmlexists patch?
>> Yes, you can probably shrink this patch down to about 20 lines.
> Updated the patch so that it will apply to head and re-worked the
> function to use the new xpath internal function.
Applied with minor corrections (improved docs, fixed regression tests,
adjusted OIDs for CVS HEAD).
regards, tom lane
In response to
pgsql-hackers by date
|Next:||From: Alexander Korotkov||Date: 2010-08-08 20:28:34|
|Subject: Re: knngist - 0.8|
|Previous:||From: Pavel Stehule||Date: 2010-08-08 18:16:45|
|Subject: Re: Initial review of xslt with no limits patch|
pgsql-rrreviewers by date
|Next:||From: Tom Lane||Date: 2010-08-09 02:29:17|
|Subject: Re: Patch review: make RAISE without arguments work like Oracle |
|Previous:||From: Mike Fowler||Date: 2010-08-07 15:38:06|
|Subject: Re: Review: Re: [PATCH] Re: [HACKERS] Adding xpath_exists function|