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

Change subject: [IMPALA-8587] Show inherited privileges with Ranger show grant
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/13673/1/fe/src/main/java/org/apache/impala/authorization/ranger/RangerImpaladAuthorizationManager.java
File 
fe/src/main/java/org/apache/impala/authorization/ranger/RangerImpaladAuthorizationManager.java:

http://gerrit.cloudera.org:8080/#/c/13673/1/fe/src/main/java/org/apache/impala/authorization/ranger/RangerImpaladAuthorizationManager.java@341
PS1, Line 341:       resourceResult.getResultRows().forEach(principal -> 
resultSet.add(principal.toResultRow()));
line too long (98 > 90)


http://gerrit.cloudera.org:8080/#/c/13673/1/tests/authorization/test_ranger.py
File tests/authorization/test_ranger.py:

http://gerrit.cloudera.org:8080/#/c/13673/1/tests/authorization/test_ranger.py@195
PS1, Line 195: t
flake8: E501 line too long (96 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/13673/1/tests/authorization/test_ranger.py@350
PS1, Line 350: )
flake8: E501 line too long (92 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5c4c9327acb12abc12d130ef5c1ace6a08ed193c
Gerrit-Change-Number: 13673
Gerrit-PatchSet: 1
Gerrit-Owner: Austin Nobis <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Tue, 18 Jun 2019 20:01:04 +0000
Gerrit-HasComments: Yes

Reply via email to