Fredy Wijaya has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13679 )

Change subject: IMPALA-8682: Add authorized proxy user/group test coverage with 
Ranger
......................................................................


Patch Set 5:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/13679/4/tests/authorization/test_authorized_proxy.py
File tests/authorization/test_authorized_proxy.py:

http://gerrit.cloudera.org:8080/#/c/13679/4/tests/authorization/test_authorized_proxy.py@195
PS4, Line 195: s
> what are these commented-out lines?
I meant to delete that. Done.


http://gerrit.cloudera.org:8080/#/c/13679/4/tests/authorization/test_authorized_proxy.py@231
PS4, Line 231:
> typo?
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If6f797600720e6432b85cac8f13afe8fa5624596
Gerrit-Change-Number: 13679
Gerrit-PatchSet: 5
Gerrit-Owner: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Comment-Date: Thu, 20 Jun 2019 17:14:38 +0000
Gerrit-HasComments: Yes

Reply via email to