Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17335 )
Change subject: [script][test] Supply --master_addresses for single master config ...................................................................... [script][test] Supply --master_addresses for single master config Supplying --master_addresses to single master configuration helps populate the 'last_known_addr' field in single master Raft configuration which is a pre-requisite for migrating to multi-master configuration. Change a59d382 previously added support to supply --master_addresses for single master configuration. https://git.io/JOD25 This change: - Updates scripts to start supplying --master_addresses by default even for single master configurations. Fixes some port numbers in a development documentation that uses the script. - Updates the default in ExternalMiniCluster to start supplying --master_addresses for single master configuration. Change-Id: I38d091b3acc607d5faf9aeb075069482af75723a Reviewed-on: http://gerrit.cloudera.org:8080/17335 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M docker/kudu-entrypoint.sh M docs/quickstartdev.adoc M src/kudu/mini-cluster/external_mini_cluster.cc M src/kudu/mini-cluster/external_mini_cluster.h M src/kudu/scripts/start_kudu.sh 5 files changed, 7 insertions(+), 8 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17335 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I38d091b3acc607d5faf9aeb075069482af75723a Gerrit-Change-Number: 17335 Gerrit-PatchSet: 2 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)
