Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17529 )
Change subject: WIP: KUDU-2612 allow system user to read list of table replicas ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/17529/1/src/kudu/integration-tests/ts_authz-itest.cc File src/kudu/integration-tests/ts_authz-itest.cc: http://gerrit.cloudera.org:8080/#/c/17529/1/src/kudu/integration-tests/ts_authz-itest.cc@705 PS1, Line 705: opts.num_tablet_servers = 3; > nit: we could also reduce this to 1, and reduce the replication factor of t The point was to have as many default settings as possible in this scenario to make sure there are less surprises in real world, but if we are sure the replication factor for the txn status table isn't bringing any surprises, I guess we could switch to RF=1 txn status table. -- To view, visit http://gerrit.cloudera.org:8080/17529 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I26f06af17e5ee85522e2ef867d41cf0f3ddbe5d5 Gerrit-Change-Number: 17529 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[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-Comment-Date: Tue, 01 Jun 2021 18:17:36 +0000 Gerrit-HasComments: Yes
