Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16321 )
Change subject: [master] KUDU-2181 Raft ChangeConfig request to add a master ...................................................................... [master] KUDU-2181 Raft ChangeConfig request to add a master This change: - Adds hidden feature flag "--master_support_change_config" off by default. - RPC changes to add a master that initiates Raft config change and responds asynchronously. - RPC changes to report back member type(VOTER/NON_VOTER) of masters - Removes the cached master_addresses in catalog manager as it's no longer static. - Updates and adds comments in MasterOptions such that it's used to fetch master addresses only during master init time as masters can be added/removed dynamically with this change. - Updates ListMasters() to look at local Raft config instead of MasterOptions as the masters can be added/removed dynamically. If the new master can be caught up from the WAL then the master gets promoted to VOTER else it remains as NON_VOTER without master tablet copying support. Change-Id: I0ac7e6e55220bcb01cad0fa386daaf656258088c Reviewed-on: http://gerrit.cloudera.org:8080/16321 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Bankim Bhavsar <[email protected]> --- M src/kudu/common/wire_protocol.proto M src/kudu/consensus/raft_consensus.cc M src/kudu/consensus/raft_consensus.h M src/kudu/master/CMakeLists.txt M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h A src/kudu/master/dynamic_multi_master-test.cc M src/kudu/master/master.cc M src/kudu/master/master.h M src/kudu/master/master.proto M src/kudu/master/master_options.cc M src/kudu/master/master_options.h M src/kudu/master/master_path_handlers.cc M src/kudu/master/master_service.cc M src/kudu/master/master_service.h M src/kudu/master/mini_master.cc M src/kudu/master/sys_catalog.cc M src/kudu/mini-cluster/external_mini_cluster.cc M src/kudu/mini-cluster/external_mini_cluster.h M src/kudu/tools/tool_action_master.cc 20 files changed, 987 insertions(+), 56 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Bankim Bhavsar: Verified -- To view, visit http://gerrit.cloudera.org:8080/16321 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0ac7e6e55220bcb01cad0fa386daaf656258088c Gerrit-Change-Number: 16321 Gerrit-PatchSet: 9 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) Gerrit-Reviewer: Tidy Bot (241)
