Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12813 )
Change subject: IMPALA-8326: Fix CreateDropRoleStmt::toSql ...................................................................... Patch Set 5: Code-Review+2 Thanks for fixing this and added a missing test. LGTM. -- To view, visit http://gerrit.cloudera.org:8080/12813 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I25e4e1f680a6992779a47b5e6ad5092f27ba390a Gerrit-Change-Number: 12813 Gerrit-PatchSet: 5 Gerrit-Owner: Austin Nobis <ano...@cloudera.com> Gerrit-Reviewer: Austin Nobis <ano...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Comment-Date: Wed, 20 Mar 2019 18:47:29 +0000 Gerrit-HasComments: No