Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 )
Change subject: [location_awareness] Add location info in ksck report ...................................................................... Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/11422/8/src/kudu/client/client.cc File src/kudu/client/client.cc: http://gerrit.cloudera.org:8080/#/c/11422/8/src/kudu/client/client.cc@574 PS8, Line 574: "" > My concern is that there are half-baked entities of KuduTabletServer becaus Will and I talked about this a bit in the morning. It seems we are going to introduce location-related stuff into TSInfoPB. Once it's there, it will be possible to get information on the tablet server locations out of replicas' locations. That will also be inline with the further work on the C++ and Java clients to make them location-aware. -- To view, visit http://gerrit.cloudera.org:8080/11422 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ideff2dd4975c99a1135002624debbbb2620fb95c Gerrit-Change-Number: 11422 Gerrit-PatchSet: 8 Gerrit-Owner: Fengling Wang <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Fengling Wang <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Tue, 09 Oct 2018 17:55:52 +0000 Gerrit-HasComments: Yes
