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

Re: [PATCH] Add XMLEXISTS function from the SQL/XML standard

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Mike Fowler <mike(at)mlfowler(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Erik Rijkers <er(at)xs4all(dot)nl>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Add XMLEXISTS function from the SQL/XML standard
Date: 2010-05-28 01:09:46
Message-ID: 201005280109.o4S19k826814@momjian.us (view raw or flat)
Thread:
Lists: pgsql-hackers
Mike Fowler wrote:
> Tom Lane wrote:
> > Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> >   
> >> On Tue, May 25, 2010 at 1:09 PM, Mike Fowler <mike(at)mlfowler(dot)com> wrote:
> >>     
> >>>> We're unlikely to accept this patch if it changes the minimum version
> >>>> of libxml2 required to compile PostgreSQL
> >>>>         
> >>> Why? 2.6.27 is almost 4 years old.
> >> At a minimum, I think it's fair to say that the burden is on you to
> >> justify what it's worth bumping the version number.
> >>     
> >
> > Yes.  Increasing the minimum required version of some library is a Big
> > Deal, we don't do it on a whim.  And we definitely don't do it just
> > because it's old.
> >
> > 			regards, tom lane
> >
> >   
> 
> OK, I consider myself suitably educated/chastised. I now understand why 
> a version bump is such a big deal. Your objections are all reasonable, I 
> suppose I'm just used to living on the bleeding edge of everything. 
> Consequently I have changed the code to produce the same result in a 
> different way without using the new function. I've down-graded my 
> version to 2.6.26 and it all compiles cleanly. Please find attached my 
> revised patch, and thanks all for your advise.

FYI, it is often good to add a comment in the C code about why you
didn't use the new XML function so if the issue comes up again, we know
why, and in 10 years, we can use it.  ;-)

-- 
  Bruce Momjian  <bruce(at)momjian(dot)us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

In response to

pgsql-hackers by date

Next:From: Tatsuo IshiiDate: 2010-05-28 01:10:07
Subject: Re: pg_trgm
Previous:From: Tatsuo IshiiDate: 2010-05-28 01:04:20
Subject: Re: pg_trgm

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