[kudu-CR] client: prefer tservers in the same location when multiple are local

["Andrew Wong (Code Review)" <ge...@cloudera.org>]

Hello Alexey Serbin, Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/15794

to look at the new patch set (#2).

Change subject: client: prefer tservers in the same location when multiple are local
......................................................................

client: prefer tservers in the same location when multiple are local

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/2
-- 
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: newpatchset
Gerrit-Change-Id: I6a0dbac073ba8a9f80a3f175157a09d06037b0df
Gerrit-Change-Number: 15794
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)