Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17401 )
Change subject: [master] KUDU-2181 Unhide master change config flag & other improvements ...................................................................... [master] KUDU-2181 Unhide master change config flag & other improvements This change: - Unhides and turns the master Raft change config flag ON by default - Updates the description of flags that are not passed to the new master - Uses sync logging on bringing up the new master to help debug any issues in startup of the new master - Updates the wait timeout while bringing up a new master to account for the higher ntp wait timeout. This timeout is also used for checking whether the new master's system catalog has caught up from WAL. So this'll bump up timeout for that case as well. Thought about introducing separate wait timeout for that case but felt like exposing too many knobs and user shouldn't care whether time is being taken for master bringup or catchup from WAL. Change-Id: Id1fc1dc5985601158ae58d5d190a60c1e542ea1d Reviewed-on: http://gerrit.cloudera.org:8080/17401 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Bankim Bhavsar <[email protected]> --- M src/kudu/master/dynamic_multi_master-test.cc M src/kudu/master/master_service.cc M src/kudu/tools/tool_action_master.cc 3 files changed, 27 insertions(+), 23 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Bankim Bhavsar: Verified -- To view, visit http://gerrit.cloudera.org:8080/17401 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id1fc1dc5985601158ae58d5d190a60c1e542ea1d Gerrit-Change-Number: 17401 Gerrit-PatchSet: 4 Gerrit-Owner: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
