Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16113 )
Change subject: KUDU-3090 Restrict changing ownership of a table ...................................................................... Patch Set 14: (1 comment) http://gerrit.cloudera.org:8080/#/c/16113/14/src/kudu/integration-tests/master_authz-itest.cc File src/kudu/integration-tests/master_authz-itest.cc: http://gerrit.cloudera.org:8080/#/c/16113/14/src/kudu/integration-tests/master_authz-itest.cc@1099 PS14, Line 1099: { : { : &MasterAuthzITestBase::ChangeOwner, : &MasterAuthzITestBase::GrantAllWithGrantTablePrivilege, : "ChangeOwner", : }, : kDatabaseName, : kTableName, : }, We should have test cases for the following behaviors too: - the user has ALL but isn't a delegate admin - the user has METADATA (or some other non-ALL privilege) and is a delegate admin - the user is the owner -- To view, visit http://gerrit.cloudera.org:8080/16113 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I75a8b24364572a84f93826ad670c543abd407bb1 Gerrit-Change-Number: 16113 Gerrit-PatchSet: 14 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Fri, 10 Jul 2020 18:21:44 +0000 Gerrit-HasComments: Yes
