Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16071 )
Change subject: KUDU-3090 Add delegate admin privilege ...................................................................... Patch Set 4: > Patch Set 3: > > (3 comments) > > Perhaps consider separating the delegate admin part of this patch from the > ownership part. E.g. we could implement the delegate admin part of this, > adding back the CreateTable privileges check (i.e. ALL WITH GRANT / ALL && > ADMIN > ALL + delegate admin), and add unit tests for the RangerAuthzProvider. Then, > once the rest of ownership lands, we can add more end-to-end tests with the > master. > > That way we can get the more controversial "admin" bits squared away > separately from the native ownership changes. Sounds good, moved it to the next patch. -- To view, visit http://gerrit.cloudera.org:8080/16071 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If8ba018dac568a1ab74cf2d5657221579636ac1c Gerrit-Change-Number: 16071 Gerrit-PatchSet: 4 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Thu, 25 Jun 2020 15:15:46 +0000 Gerrit-HasComments: No
