Hello Alexey Serbin,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/15794
to review the following change.
Change subject: client: prefer local tservers in the same location
......................................................................
client: prefer local tservers in the same location
be68ce8 introduced the behavior that mini servers that are local to a
client are treated as local, even on Linux. As such, a location-aware
mini-cluster would not honor client location awareness, since all
servers would be considered local.
While this surfaced as flakiness of location_awareness-itest, this is an
actual product deficiency: if two servers are colocated, but are
assigned different locations, those locations should be honored when
picking tablet servers.
Without this patch, location_assignment-itest is ~24% flaky. With it,
this passed in debug mode 300/300 times.
Change-Id: I6a0dbac073ba8a9f80a3f175157a09d06037b0df
---
M src/kudu/client/client-internal.cc
M src/kudu/integration-tests/location_assignment-itest.cc
2 files changed, 14 insertions(+), 18 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/94/15794/1
--
To view, visit http://gerrit.cloudera.org:8080/15794
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6a0dbac073ba8a9f80a3f175157a09d06037b0df
Gerrit-Change-Number: 15794
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
