Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11115 )
Change subject: [location_awareness] Assign locations to registering tablet servers ...................................................................... [location_awareness] Assign locations to registering tablet servers This patch introduces a location field maintained by the master for each tablet server. The master determines the value of this field whenever a tablet server registers. It does this by using an external command, specified with the flag --location_mapping_cmd, to produce a location from the hostname or IP address of the tablet server. To help with ad hoc testing, I also added the location field to the /tablet-servers web page, and fixed a small oversight where the table of tablet servers wasn't sorted, so its order changed depending on the order tablet servers first registered in. I also altered the DATA_FILES CMake function so that data files copied to the build directory or to slaves by dist-test are executable as well as readable. This was necessary for the new TSDescriptor test which tests location assignment. Change-Id: I5eb98823ab7b3b8141b8630196c29c1ebf8e6878 Reviewed-on: http://gerrit.cloudera.org:8080/11115 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M CMakeLists.txt M src/kudu/master/CMakeLists.txt M src/kudu/master/master-test.cc M src/kudu/master/master_path_handlers.cc A src/kudu/master/testdata/first_argument.sh A src/kudu/master/ts_descriptor-test.cc M src/kudu/master/ts_descriptor.cc M src/kudu/master/ts_descriptor.h M www/tablet-servers.mustache 9 files changed, 342 insertions(+), 12 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11115 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5eb98823ab7b3b8141b8630196c29c1ebf8e6878 Gerrit-Change-Number: 11115 Gerrit-PatchSet: 10 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Fengling Wang <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <[email protected]>
