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

Change subject: IMPALA-8225: Add Ranger support for grant/revoke privilege 
to/from user
......................................................................


Patch Set 8:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12769/8/tests/authorization/test_ranger.py
File tests/authorization/test_ranger.py:

http://gerrit.cloudera.org:8080/#/c/12769/8/tests/authorization/test_ranger.py@45
PS8, Line 45:
            :     admin_client.execute("grant select on database functional to 
user {0}".format(user),
            :                          user="admin")
            :     # TODO: IMPALA-8293 use refresh authorization
            :     time.sleep(30)
            :     user_client.execute("show tables in functional", user=user)
            :     admin_client.execute("revoke select on database functional 
from user {0}"
            :                          .format(user), user="admin")
can you use execute_query_expect_success here?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6ee97bf41546d63385026c0e2b19545565402462
Gerrit-Change-Number: 12769
Gerrit-PatchSet: 8
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, 27 Mar 2019 16:35:54 +0000
Gerrit-HasComments: Yes

Reply via email to