Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17490 )
Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments ...................................................................... Patch Set 3: > Thank you for fixing this. > > I guess we will also need to make sure TxnManager is able to create > new ranges and write into tables/tablets. I'll take a look at that > -- we can add corresponding permissions for methods in > TabletServerService in a separate changelist. I took a look at that and it seems TxnManager uses regular MasterService::AlterTable() RPC which already has "AuthorizeClientOrServiceUser" -- To view, visit http://gerrit.cloudera.org:8080/17490 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4622586fc27b5b67005bf023e4fbaebaf5454ad0 Gerrit-Change-Number: 17490 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Mon, 24 May 2021 16:58:41 +0000 Gerrit-HasComments: No
