From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Add ALTER TABLESPACE ... MOVE command |
Date: | 2014-04-13 05:10:15 |
Message-ID: | 20140413051015.GN2556@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
* Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
> Ick. That's just plain sloppy. Please create a separate production,
> *and* a separate comment header.
Done.
> Commit d86d51a95 was pretty damn awful in this regard as well, but
> let's clean them both up, not make it worse.
Yeah, I think I noticed this in passing but probably did the same as
Robert and figured "oh, I'm just playing with this, I'll go back and
clean it up later.." and then promptly forgot about it 'cause it worked
just fine.
> Existing precedent would suggest inventing two new productions named the
> same as the parse node types they produce, viz AlterTableSpaceMoveStmt
> and AlterTableSpaceOptionsStmt.
This felt a bit like overkill to me for these few, so I just did them
under a single 'AlterTblSpcStmt'. I'm not against going back and
changing it if others feel differently.
Thanks,
Stephen
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2014-04-13 12:40:33 | Re: pgsql: Add ANALYZE into regression tests |
Previous Message | Stephen Frost | 2014-04-13 05:04:53 | pgsql: Make a dedicated AlterTblSpcStmt production |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2014-04-13 05:16:51 | Re: proposal: interprocess EXPLAIN PID |
Previous Message | David Rowley | 2014-04-13 04:59:28 | Patch to fix a couple of compiler warnings from 80a5cf64 |