Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Bankim Bhavsar,

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

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

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

Change subject: ksck: print tablet server states
......................................................................

ksck: print tablet server states

This adds a table of tserver states to the ksck output:

...
Tablet Server States
              Server              |      State
----------------------------------+------------------
 767c081ea5564c9a8ed8615c00658e1b | MAINTENANCE_MODE

Tablet Server Summary
...

When there are no special tablet server states, the section doesn't get
printed.

I wanted to reuse some of the client code to list tablet servers but didn't
want to change the public interface, so this patch also rejiggers the
ListTabletServers client logic into the client data class.

Change-Id: I4cec8c85a08bcb4f7be9e98a34d4b5ae2f05f3be
---
M src/kudu/client/client-internal.cc
M src/kudu/client/client-internal.h
M src/kudu/client/client.cc
M src/kudu/client/client.h
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/kudu-tool-test.cc
10 files changed, 136 insertions(+), 27 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4cec8c85a08bcb4f7be9e98a34d4b5ae2f05f3be
Gerrit-Change-Number: 14436
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)

Reply via email to