Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13346 )

Change subject: IMPALA-8476: Replace Sentry admin check workaround with proper 
Sentry API
......................................................................


Patch Set 10:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/13346/10/fe/src/main/java/org/apache/impala/authorization/sentry/SentryPolicyService.java
File 
fe/src/main/java/org/apache/impala/authorization/sentry/SentryPolicyService.java:

http://gerrit.cloudera.org:8080/#/c/13346/10/fe/src/main/java/org/apache/impala/authorization/sentry/SentryPolicyService.java@534
PS10, Line 534:
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/13346/10/fe/src/main/java/org/apache/impala/authorization/sentry/SentryPolicyService.java@537
PS10, Line 537:         SentryServiceClient client = new SentryServiceClient();
tab used for whitespace


http://gerrit.cloudera.org:8080/#/c/13346/10/fe/src/main/java/org/apache/impala/authorization/sentry/SentryPolicyService.java@538
PS10, Line 538:         try {
tab used for whitespace


http://gerrit.cloudera.org:8080/#/c/13346/10/fe/src/main/java/org/apache/impala/authorization/sentry/SentryProxy.java
File fe/src/main/java/org/apache/impala/authorization/sentry/SentryProxy.java:

http://gerrit.cloudera.org:8080/#/c/13346/10/fe/src/main/java/org/apache/impala/authorization/sentry/SentryProxy.java@387
PS10, Line 387:         return 
sentryPolicyService_.isSentryAdmin(requestingUser);
tab used for whitespace



--
To view, visit http://gerrit.cloudera.org:8080/13346
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5a27140d401494bc372ad0da96ada57bda94cd82
Gerrit-Change-Number: 13346
Gerrit-PatchSet: 10
Gerrit-Owner: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Wed, 26 Jun 2019 01:19:42 +0000
Gerrit-HasComments: Yes

Reply via email to