Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10886 )
Change subject: IMPALA-7216: Fix SQL generated by CreateViewStmt & AlterViewStmt ...................................................................... IMPALA-7216: Fix SQL generated by CreateViewStmt & AlterViewStmt The toSql functions in CreateViewStmt and AlterViewStmt generated invalid SQL by appending types to column definitions. This change appends just the column names to fix it. Testing: Added tests to ToSqlTest to verify it. Change-Id: Ie2a834d4d8d11ba18262d2828f4829c80ed05190 Reviewed-on: http://gerrit.cloudera.org:8080/10886 Reviewed-by: Bikramjeet Vig <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/analysis/AlterViewStmt.java M fe/src/main/java/org/apache/impala/analysis/CreateOrAlterViewStmtBase.java M fe/src/main/java/org/apache/impala/analysis/CreateViewStmt.java M fe/src/test/java/org/apache/impala/analysis/ToSqlTest.java 4 files changed, 81 insertions(+), 4 deletions(-) Approvals: Bikramjeet Vig: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10886 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie2a834d4d8d11ba18262d2828f4829c80ed05190 Gerrit-Change-Number: 10886 Gerrit-PatchSet: 8 Gerrit-Owner: Pooja Nilangekar <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Pooja Nilangekar <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
