Hello Will Berkeley, Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11679
to look at the new patch set (#2).
Change subject: Add ts location to both client and internal client
......................................................................
Add ts location to both client and internal client
I added the location field into KuduTabletServer::Data.
When initializing KuduTabletServer::Data, we have three different
sources to get the location: RemoteTabletServer, TSInfoPB, and
ListTabletServersResponsePB - so far the location is missing in
RemoteTabletServer and TSInfoPB.
So I added the location into them in order to ensure the consistency
of KuduTabletServer::Data.
Change-Id: Ib92cc6806073d32c859ae44ff803abb37cac99ac
---
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/meta_cache.cc
M src/kudu/client/meta_cache.h
M src/kudu/client/scan_token-internal.cc
M src/kudu/client/tablet_server-internal.cc
M src/kudu/client/tablet_server-internal.h
M src/kudu/master/master.proto
8 files changed, 31 insertions(+), 7 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/79/11679/2
--
To view, visit http://gerrit.cloudera.org:8080/11679
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib92cc6806073d32c859ae44ff803abb37cac99ac
Gerrit-Change-Number: 11679
Gerrit-PatchSet: 2
Gerrit-Owner: Fengling Wang <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Fengling Wang <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley <[email protected]>
