Hello Will Berkeley, Dan Burkert, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/9787
to review the following change.
Change subject: KUDU-2364 Add extra check in ksck for tserver ID
......................................................................
KUDU-2364 Add extra check in ksck for tserver ID
ksck did not validate tablet server ID when checking connectivity.
Whenever the TabletServer was nuked and readded, ksck would report the
connection was successful to the old TabletServer.
Now it will report an error like below:
Connected to the Master
WARNING: Unable to connect to Tablet Server
71167ec2051f4c1bb96b6a2cb54cc31c (va1022.halxg.cloudera.com:7050):
Invalid argument: ListTablets: Wrong destination UUID requested. Local
UUID: d92197fa2f034b33aa0bf998c41f637b. Requested UUID:
71167ec2051f4c1bb96b6a2cb54cc31c
WARNING: Fetched info from 1 Tablet Servers, 1 weren't reachable
The cluster doesn't have any matching tables
==================
Errors:
==================
error fetching info from tablet servers: Could not gather complete
information from all tablet servers
Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526
---
M src/kudu/tools/ksck_remote.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
3 files changed, 13 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/9787/1
--
To view, visit http://gerrit.cloudera.org:8080/9787
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526
Gerrit-Change-Number: 9787
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
