Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 )
Change subject: [tools] command to reset authz cache in kudu CLI ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/13139/5/src/kudu/tools/tool_action_master.cc File src/kudu/tools/tool_action_master.cc: http://gerrit.cloudera.org:8080/#/c/13139/5/src/kudu/tools/tool_action_master.cc@187 PS5, Line 187: if (resp_master_addrs.size() != master_addresses.size()) { > It seemed to me that the consensus was that we want to make the reset opera Right, I guess I assumed the consensus was we'd just have the operator run the tool multiple times, one for each master, as they would for other `kudu master` tooling. --force flag seems ok too. Maybe we should add this behavior to set_flag as well. -- To view, visit http://gerrit.cloudera.org:8080/13139 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If49f3ebdea22ea0df3aaec313b4db949dc834943 Gerrit-Change-Number: 13139 Gerrit-PatchSet: 5 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sat, 04 May 2019 01:48:07 +0000 Gerrit-HasComments: Yes