From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Merlin Moncure <mmoncure(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: idea: allow AS label inside ROW constructor |
Date: | 2014-10-22 17:09:52 |
Message-ID: | CAFj8pRBRdRrt6jfd4BA_diw+sAAOxY69oJUO=LJJ-vWSH9maDw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2014-10-22 18:35 GMT+02:00 Merlin Moncure <mmoncure(at)gmail(dot)com>:
> On Wed, Oct 22, 2014 at 11:21 AM, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
> wrote:
> > Hi
> >
> > with new functions row_to_json(b), there is more often usage of ROW
> > constructor. Using names in fields is relative difficult. Because ROW has
> > special clause in parser, I am thinking so we can enable labeling inside
> ROW
> > constructor
> >
> > so instead currently supported:
> >
> > select row_to_json(r) from (select 10 as a, 20 as b) r;
> >
> > users can to write:
> >
> > select row_to_json(row(10 as a,20 as b));
> >
> > labeling will be enabled "only" inside ROW constructor. I don't propose
> > enable it everywhere.
> >
> > What do you think about it?
>
> It's a neat idea -- maybe a better alternative to what I was thinking
> here:
> http://postgresql.1045698.n5.nabble.com/Support-UPDATE-table-SET-tp5823073p5823944.html
>
> Some questions:
> *) What would the parser transformation resolve to
>
row: ROW '(' expr_list ')' { $$
= $3; }
| ROW '('
')' { $$ = NIL; }
| '(' expr_list ',' a_expr ')' {
$$ = lappend($2, $4); }
;
we can replace a expr_list by target_list. I know only so it doesn't
enforce a problems with gramatic - bison doesn't raise any warning.
*) Are we ok with SQL standard
>
SQL standard doesn't think named attributes in row - so it is out of range
ANSI. But it is not in conflict with standard. "AS name" is used more in
SQL/MM, SQL/XML -- and function named parameters has different syntax
"parameter_name <= value" - I checked it against SQL99.
> *) Do you think this (or some similar variant) would work?
>
> select row_to_json(row(foo.*)) from foo;
>
It looks like independent feature and can work too - it is more natural for
user.
>
> merlin
>
From | Date | Subject | |
---|---|---|---|
Next Message | Martijn van Oosterhout | 2014-10-22 18:54:44 | Re: [PATCH] add ssl_protocols configuration option |
Previous Message | Kevin Grittner | 2014-10-22 17:07:16 | Re: Question about RI checks |