Pooja Nilangekar has posted comments on this change. ( http://gerrit.cloudera.org:8080/11043 )
Change subject: IMPALA-7218: [DOCS] Support column list in ALTER VIEW ...................................................................... Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/11043/3/docs/topics/impala_alter_view.xml File docs/topics/impala_alter_view.xml: http://gerrit.cloudera.org:8080/#/c/11043/3/docs/topics/impala_alter_view.xml@82 PS3, Line 82: c3 Please remove this. Since the select statement has two columns (x, s), the column definition list should also have two columns. http://gerrit.cloudera.org:8080/#/c/11043/3/docs/topics/impala_alter_view.xml@105 PS3, Line 105: Nit: Please remove the new line. http://gerrit.cloudera.org:8080/#/c/11043/3/docs/topics/impala_alter_view.xml@118 PS3, Line 118: Nit: Please remove the new line. http://gerrit.cloudera.org:8080/#/c/11043/3/docs/topics/impala_alter_view.xml@120 PS3, Line 120: Nit: Please remove the new line. -- To view, visit http://gerrit.cloudera.org:8080/11043 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I19e5cf97302a46738fd832344415fb7ad4ca0e41 Gerrit-Change-Number: 11043 Gerrit-PatchSet: 3 Gerrit-Owner: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Pooja Nilangekar <pooja.nilange...@cloudera.com> Gerrit-Comment-Date: Wed, 25 Jul 2018 18:04:35 +0000 Gerrit-HasComments: Yes
