Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17024 )
Change subject: [master] turn off client location assignment by default ...................................................................... [master] turn off client location assignment by default This patch turns off location assignment by default to clients connecting to a Kudu cluster. The assigned locations are cached, but the way how locations assignment performed is resource consuming, see [1] for details. There aren't many benefits in assigning locations to clients so far: the only nice property of a client with assigned location vs a client with no location assigned is that former case the client run from a particular location would choose tablet servers in the same location if performing scan in ReplicaSelection::CLOSEST_REPLICA mode. This is a first step towards addressing KUDU-3212. [1] https://issues.apache.org/jira/browse/KUDU-3212 Change-Id: I78474ced0a0129b3f2b1add55f6f908a136106d0 Reviewed-on: http://gerrit.cloudera.org:8080/17024 Tested-by: Kudu Jenkins Reviewed-by: Hao Hao <[email protected]> Reviewed-by: Grant Henke <[email protected]> --- M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java M src/kudu/client/client-test.cc M src/kudu/integration-tests/location_assignment-itest.cc M src/kudu/master/master-test.cc M src/kudu/master/master_service.cc 5 files changed, 19 insertions(+), 4 deletions(-) Approvals: Kudu Jenkins: Verified Hao Hao: Looks good to me, but someone else must approve Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17024 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I78474ced0a0129b3f2b1add55f6f908a136106d0 Gerrit-Change-Number: 17024 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
