Fang-Yu Rao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16837 )

Change subject: IMPALA-10211 (Part 1): Add support for role-related statements
......................................................................


Patch Set 6:

(1 comment)

> Patch Set 6: Code-Review+1
>
> (1 comment)
>
> The change looks good to me in general.
> About the RANGER Jiras: I would prefer to create them before merging this and 
> mention them in comments or the commit message.

Thanks Csaba! With regard to this, I have created RANGER-3125, RANGER-3126, and 
RANGER-3127 to keep track of the issues we have found. In addition, I also 
created IMPALA-10399 regarding the issue of the test 
test_show_grant_hive_privilege() we encountered.

http://gerrit.cloudera.org:8080/#/c/16837/4/testdata/workloads/functional-query/queries/QueryTest/grant_revoke.test
File testdata/workloads/functional-query/queries/QueryTest/grant_revoke.test:

http://gerrit.cloudera.org:8080/#/c/16837/4/testdata/workloads/functional-query/queries/QueryTest/grant_revoke.test@1277
PS4, Line 1277: # Clean up the granted privileges and test roles.
> > It would be great if we had some statement like "REVOKE <privilege> ON <r
Thanks Csaba! In this regard, I will add to test_grant_revoke_with_role() a 
section that would be executed to clean up the granted privileges as well as 
the test roles in case some error occurs in the execution of the test cases in 
this file.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic2b204e62a1d8ae1932d955b4efc28be22202860
Gerrit-Change-Number: 16837
Gerrit-PatchSet: 6
Gerrit-Owner: Fang-Yu Rao <fangyu....@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Fang-Yu Rao <fangyu....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Comment-Date: Thu, 17 Dec 2020 00:37:50 +0000
Gerrit-HasComments: Yes

Reply via email to