Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/12975 )
Change subject: IMPALA-8391: [DOCS] ALTER TABLE RENAME now renames the underlying Kudu TABLE ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/12975/1/docs/topics/impala_tables.xml File docs/topics/impala_tables.xml: http://gerrit.cloudera.org:8080/#/c/12975/1/docs/topics/impala_tables.xml@302 PS1, Line 302: You can use <codeph>ALTER TABLE</codeph> to rename the Impala table > change "<codeph>ALTER TABLE</codeph>" to "<codeph>ALTER TABLE <varname>[ol The change below covers this, as well. http://gerrit.cloudera.org:8080/#/c/12975/1/docs/topics/impala_tables.xml@303 PS1, Line 303: and the underlying Kudu table. > I would phrase this more like: you can use "ALTER TABLE..." to rename the I The below change will cover this as well. http://gerrit.cloudera.org:8080/#/c/12975/1/docs/topics/impala_tables.xml@306 PS1, Line 306: You can use <codeph>ALTER TABLE <varname>impala_name</varname> SET > I would re-organize this a bit to have two distinct sections: one for renam Done -- To view, visit http://gerrit.cloudera.org:8080/12975 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I704a0623a05166e5f636b6c1063e9feffb58e65b Gerrit-Change-Number: 12975 Gerrit-PatchSet: 1 Gerrit-Owner: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com> Gerrit-Comment-Date: Wed, 10 Apr 2019 22:19:26 +0000 Gerrit-HasComments: Yes