Hello Will Berkeley, Dan Burkert, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9787
to look at the new patch set (#4).
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:
./bin/kudu cluster ksck localhost
Connected to the Master
WARNING: Unable to connect to Tablet Server
b68c8e3352ff411da5bbe177631ace5d (chargers-macbook-pro.local:7050):
Remote error: ID reported by TabletServer
(5e929ae0b09447cc9444a4b635a0d5ac) doesn't match the expected ID:
b68c8e3352ff411da5bbe177631ace5d
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
FAILED
Runtime error: ksck discovered errors
Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526
---
M src/kudu/mini-cluster/internal_mini_cluster.h
M src/kudu/server/server_base.cc
M src/kudu/server/server_base.proto
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck_remote-test.cc
M src/kudu/tools/ksck_remote.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
8 files changed, 66 insertions(+), 12 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/9787/4
--
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: newpatchset
Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526
Gerrit-Change-Number: 9787
Gerrit-PatchSet: 4
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley <[email protected]>
