Skip site navigation (1) Skip section navigation (2)

Re: Review: Re: [PATCH] Re: [HACKERS] Adding xpath_exists function

From: Mike Fowler <mike(at)mlfowler(dot)com>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: David Fetter <david(at)fetter(dot)org>, Bruce Momjian <bruce(at)momjian(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Round Robin Reviewers <pgsql-rrreviewers(at)postgresql(dot)org>
Subject: Re: Review: Re: [PATCH] Re: [HACKERS] Adding xpath_exists function
Date: 2010-08-06 08:04:15
Message-ID: 4C5BC1FF.4090604@mlfowler.com (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-rrreviewers
On 06/08/10 05:38, Peter Eisentraut wrote:
> On tis, 2010-07-27 at 16:33 -0700, David Fetter wrote:
>    
>> * Do we already have it?
>>
>>      Not really.  There are kludges to accomplish these things, but
>>      they're available mostly in the sense that a general-purpose
>>      language allows you to write code to do anything a Turing machine
>>      can do.
>>      
> I think this has been obsoleted by the xmlexists patch

In many ways yes. The only surviving difference is that xpath_exists has 
support for namespaces and xmlexists does not as the grammar expects 
namespaces to be handled in the xquery. So if people expect namespace 
support to be useful that having both functions is useful until I (or 
someone who works faster than me) get xquery going.

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?

Regards,

-- 
Mike Fowler
Registered Linux user: 379787


In response to

Responses

pgsql-hackers by date

Next:From: Boxuan ZhaiDate: 2010-08-06 08:23:58
Subject: Re: MERGE Specification
Previous:From: Naveed AlamDate: 2010-08-06 07:56:01
Subject: Re: Proposal / proof of concept: Triggers on VIEWs

pgsql-rrreviewers by date

Next:From: Kevin GrittnerDate: 2010-08-06 14:31:13
Subject: Re: CommitFest 2010-07 week three progress report
Previous:From: Peter EisentrautDate: 2010-08-06 04:38:12
Subject: Re: Review: Re: [PATCH] Re: [HACKERS] Adding xpath_exists function

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group