From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Jim Nasby <jnasby(at)pervasive(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Susanne Ebrecht <miracee(at)miracee(dot)de>, pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org |
Subject: | Re: [HACKERS] extension for sql update |
Date: | 2006-07-27 22:02:27 |
Message-ID: | 200607272202.k6RM2RB09421@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Jim Nasby wrote:
> On Jul 27, 2006, at 7:30 AM, Tom Lane wrote:
> > Susanne Ebrecht <miracee(at)miracee(dot)de> writes:
> >> ... We could provide the mixed update syntax and leave the
> >> typed row value expression for the next release. Do you agree?
> >
> > I don't really see the point --- the patch won't provide any new
> > functionality in anything like its current form, because you can
> > always just write the separate expressions in the simple one to
> > one way. If we do offer the row-on-the-left syntax then people
> > will try to put sub-selects on the right, and won't get anything
> > beyond an unhelpful "syntax error" message. So my vote would be
> > to leave it alone until we have a more complete implementation.
>
> While the patch doesn't provide any new functionality, I would still
> welcome it simply because I find it a lot easier and cleaner to group
> fields together when updating multiple fields at once.
>
> Even if we would have to rip this patch back out in order to fully
> support the spec, we've got a (mostly) working patch right now, and
> it sounds like it would take minimal effort to finish it.
>
> In any case, it sounds like there should be a TODO item out of this.
We already had it on the TODO list, but I didn't realize about the
subselect issue. I added a sentence to clarify that:
o Allow UPDATE tab SET ROW (col, ...) = (...) for updating multiple
columns
new--> A subselect can also be used as the value source.
--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2006-07-27 22:13:45 | Re: [COMMITTERS] pgsql: another try at keeping AIX/ppc |
Previous Message | Tom Lane | 2006-07-27 21:47:40 | Re: [COMMITTERS] pgsql: another try at keeping AIX/ppc |
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2006-07-27 23:29:14 | Re: The vacuum-ignore-vacuum patch |
Previous Message | Greg Stark | 2006-07-27 21:24:35 | Re: [HACKERS] Resurrecting per-page cleaner for btree |