Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10919 )
Change subject: IMPALA-7275: Create table authorization error should not show table name ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/10919/4/fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java File fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java: http://gerrit.cloudera.org:8080/#/c/10919/4/fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java@124 PS4, Line 124: Since we don't get back the modified : // authorizable list, we need to check again and modify the error nit: I think this comment is no longer relevant. -- To view, visit http://gerrit.cloudera.org:8080/10919 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6711a744541abd5ff26574974ba8517b6e51c453 Gerrit-Change-Number: 10919 Gerrit-PatchSet: 4 Gerrit-Owner: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Comment-Date: Wed, 11 Jul 2018 17:07:11 +0000 Gerrit-HasComments: Yes