Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-04-04 Thread Peter Eisentraut
Am Mittwoch, 4. April 2007 14:43 schrieb Nikolay Samokhvalov: Why do we even need to support xpath on fragments? Why not? I find it useful and convenient. Well, rather than inventing bogus root wrapper elements, why not let users call xmlelement() to produce the wrapper element themselves?

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-04-04 Thread Nikolay Samokhvalov
On 4/4/07, Peter Eisentraut [EMAIL PROTECTED] wrote: Am Mittwoch, 4. April 2007 14:43 schrieb Nikolay Samokhvalov: Why do we even need to support xpath on fragments? Why not? I find it useful and convenient. Well, rather than inventing bogus root wrapper elements, why not let users call

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-04-04 Thread Nikolay Samokhvalov
On 3/23/07, Peter Eisentraut [EMAIL PROTECTED] wrote: Andrew Dunstan wrote: Would it be better to use some more unlikely name for the dummy root element used to process fragments than x ? Why do we even need to support xpath on fragments? Why not? I find it useful and convenient. -- Best

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-03-22 Thread Bruce Momjian
Applying newest version of this patch now; still needs documentation. --- Nikolay Samokhvalov wrote: On 3/5/07, Nikolay Samokhvalov [EMAIL PROTECTED] wrote: On 3/4/07, Nikolay Samokhvalov [EMAIL PROTECTED] wrote:

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-03-22 Thread Peter Eisentraut
Andrew Dunstan wrote: Would it be better to use some more unlikely name for the dummy root element used to process fragments than x ? Why do we even need to support xpath on fragments? -- Peter Eisentraut http://developer.postgresql.org/~petere/ ---(end of

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-03-18 Thread Nikolay Samokhvalov
On 3/5/07, Nikolay Samokhvalov [EMAIL PROTECTED] wrote: On 3/4/07, Nikolay Samokhvalov [EMAIL PROTECTED] wrote: I'll fix these issues and extend the patch with resgression tests and docs for xpath_array(). I'll resubmit it very soon. Here is a new version of the patch. I didn't change any

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-03-18 Thread Nikolay Samokhvalov
On 3/17/07, Andrew Dunstan [EMAIL PROTECTED] wrote: In principle I am in favor of the patch. Would it be better to use some more unlikely name for the dummy root element used to process fragments than x ? Perhaps even something in a special namespace? I did think about it, but I didn't find

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-03-17 Thread Nikolay Samokhvalov
What about it? W/o this not large patch XML functionality in 8.3 will be weak... Will it be accepted? On 3/5/07, Nikolay Samokhvalov [EMAIL PROTECTED] wrote: On 3/4/07, Nikolay Samokhvalov [EMAIL PROTECTED] wrote: I'll fix these issues and extend the patch with resgression tests and docs for

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-03-17 Thread Andrew Dunstan
Nikolay Samokhvalov wrote: What about it? W/o this not large patch XML functionality in 8.3 will be weak... Will it be accepted? In principle I am in favor of the patch. Would it be better to use some more unlikely name for the dummy root element used to process fragments than x ?

Re: [HACKERS] [PATCHES] xpath_array with namespaces support

2007-03-17 Thread Andrew Dunstan
Nikolay Samokhvalov wrote: On 3/17/07, Andrew Dunstan [EMAIL PROTECTED] wrote: In principle I am in favor of the patch. Would it be better to use some more unlikely name for the dummy root element used to process fragments than x ? Perhaps even something in a special namespace? I did