Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong,

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

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

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

Change subject: [tools] Add version info and check to ksck
......................................................................

[tools] Add version info and check to ksck

This adds a new version summary table, similar to the one on the
master web ui for tablet servers, that summarizes the versions of Kudu
running in the cluster (both masters and tablet servers). ksck will
issue a warning if there are multiple versions. The version info is
available as part of the server health summaries in the KsckResults, so
clients of the Ksck class can use this information. Version info is
plumbed into the JSON output as well.

An example table and warning:

Version Summary
    Version     |                                                  Servers
----------------+-----------------------------------------------------------------------------------------------------
 1.6.0          | [email protected], [email protected], 
[email protected], and 1 other server(s)
 1.7.0          | [email protected]
 1.8.0-SNAPSHOT | [email protected]

==================
Warnings:
==================
version check error: not all servers are running the same version: 3 different 
versions were seen

Change-Id: I4a2bc7138f074ab8d74f21ace2eb2b8018c53f50
---
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck.cc
M src/kudu/tools/ksck.h
M src/kudu/tools/ksck_remote.cc
M src/kudu/tools/ksck_results.cc
M src/kudu/tools/ksck_results.h
M src/kudu/tools/tool.proto
7 files changed, 118 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/79/10579/3
--
To view, visit http://gerrit.cloudera.org:8080/10579
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4a2bc7138f074ab8d74f21ace2eb2b8018c53f50
Gerrit-Change-Number: 10579
Gerrit-PatchSet: 3
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>

Reply via email to