Hello Fengling Wang, Alexey Serbin, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11115 to look at the new patch set (#8). 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 --- M CMakeLists.txt M src/kudu/master/CMakeLists.txt 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 8 files changed, 304 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/11115/8 -- 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: newpatchset Gerrit-Change-Id: I5eb98823ab7b3b8141b8630196c29c1ebf8e6878 Gerrit-Change-Number: 11115 Gerrit-PatchSet: 8 Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Fengling Wang <fw...@cloudera.com> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>