Groups
Groups
Sign in
Groups
Groups
hypertable-code-review
Conversations
About
Send feedback
Help
[Issue 340] Added "ALTER TABLE CHANGE" feature to rename column family names.
0 views
Skip to first unread message
Sanjit Jhala
unread,
Oct 2, 2009, 6:29:51 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
[Issue 340] Added "ALTER TABLE CHANGE" feature to rename column family names.
-Modified hypertable shell regression test.
-Updated documentation with LDI option "IGNORE_UNKNOWN_CFS" and "ALTER TABLE
CHANGE".
http://github.com/sjhalaz/hypertable/commit/27d8c377b0667780df15beffdab5ee7e0cda1e01
-Sanjit
Luke
unread,
Oct 2, 2009, 7:29:38 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
CHANGE sounds a little to vague, RENAME would be better.
Sanjit Jhala
unread,
Oct 2, 2009, 7:35:38 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
I agree, I went with CHANGE because thats what MySql uses for column renaming.
http://php.about.com/od/mysqlcommands/g/mysql_change.htm
-Sanjit
Luke
unread,
Oct 2, 2009, 7:40:18 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
Well, PostgreSQL use RENAME
http://www.postgresql.org/docs/8.1/static/ddl-alter.html
, and it's
known for consistent syntax, unlike MySQL :)
Sanjit Jhala
unread,
Oct 2, 2009, 7:54:57 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
Sounds good. I'll *change* it to rename
Sanjit Jhala
unread,
Oct 2, 2009, 8:03:01 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
While I'm at it, maybe I should be something like:
"ALTER TABLE foo RENAME COLUMN FAMILY (old, new)"
to distinguish from potential future changes like RENAME ACCESS GROUP, RENAME TABLE etc.
-Sanjit
Luke
unread,
Oct 2, 2009, 8:03:43 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
Sounds good.
Sanjit Jhala
unread,
Oct 2, 2009, 8:18:33 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
Updated commit
http://github.com/sjhalaz/hypertable/commit/3249cafbf86cb367725c2015fc10b07ba5c9e062
Luke
unread,
Oct 2, 2009, 8:22:55 PM
10/2/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
Looks like forgot to update the docs (alter-table.md and HqlHelpText.cc etc.) :)
Sanjit Jhala
unread,
Oct 4, 2009, 2:18:40 PM
10/4/09
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to hypertable-...@googlegroups.com
http://github.com/sjhalaz/hypertable/commit/20dffe401229087c07fa9d561faf921bde37a6ba
Reply all
Reply to author
Forward
0 new messages