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

Re: patch: function xmltable

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Craig Ringer <craig(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: patch: function xmltable
Date: 2017-01-12 15:02:07
Message-ID: CAFj8pRBTmHsJ-mU4n75JhuS8-HK1YyDPYF6LQbPd3D1fDwNKXw@mail.gmail.com (view raw, whole thread or download thread mbox)
Thread:
Lists: pgsql-hackers
Hi

2017-01-11 22:53 GMT+01:00 Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>:

> Alvaro Herrera wrote:
>
> > The more I look at this, the less I like using NameArgExpr for
> > namespaces.  It looks all wrong to me, and it causes ugly code all over.
> > Maybe I just need to look at it a bit longer.
>
> I think it'd be cleaner to use ResTarget for the namespaces, like
> xml_attribute_el does, and split the names from actual exprs in the same
> way.  So things like ExecInitExpr become simpler because you just
> recurse to initialize the list, without having to examine each element
> individually.  tabexprInitialize can just do forboth().
>
> The main reason I don't like abusing NamedArgExpr is that the whole
> comment that explains it becomes a lie.
>

I used your proposed way based on Restarget

Updated patch attached.

Regards

Pavel


>
> --
> Álvaro Herrera                https://www.2ndQuadrant.com/
> PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
>

Attachment: xmltable-27.patch
Description: text/x-patch (162.6 KB)

In response to

Responses

pgsql-hackers by date

Next:From: Peter EisentrautDate: 2017-01-12 15:06:51
Subject: pg_upgrade vs. pg_ctl stop -m fast
Previous:From: Jesper PedersenDate: 2017-01-12 14:38:53
Subject: Re: Retiring from the Core Team

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