Austin Nobis has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12817
Change subject: IMPALA-8328: Add missing ToSql test cases for authorization statements ...................................................................... IMPALA-8328: Add missing ToSql test cases for authorization statements ToSql test cases were added for the following authorization statement classes: ShowRolesStmt, ShowGrantPrincipalStmt, GrantRevokeRoleStmt, GrantRevokePrivStmt. In addition, the ShowGrantPrincipalStmt::toSql and GrantRevokeRoleStmt::toSql statements were found to be incorrect during testing and were updated to align with the grammar. Testing: - Ran all FE tests - Added new tests to ToSqlTest for the listed statements Change-Id: I828d0459b6c92f7f15006e2353ce108093aa9dab --- M fe/src/main/java/org/apache/impala/analysis/GrantRevokeRoleStmt.java M fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java M fe/src/main/java/org/apache/impala/analysis/ShowGrantPrincipalStmt.java M fe/src/test/java/org/apache/impala/analysis/ToSqlTest.java 4 files changed, 109 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/17/12817/1 -- To view, visit http://gerrit.cloudera.org:8080/12817 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I828d0459b6c92f7f15006e2353ce108093aa9dab Gerrit-Change-Number: 12817 Gerrit-PatchSet: 1 Gerrit-Owner: Austin Nobis <ano...@cloudera.com>