Will Berkeley has posted comments on this change. Change subject: [tools] Add summary-only mode to ksck ......................................................................
Patch Set 2: (1 comment) I realized there's quite a bit more work to do to get machine-readable output from ksck, since there's also messages printed from the calling tool action code, and it'd be nice to integrate checksum results and more info into the json. Let me work some more on this and post a more complete change later. http://gerrit.cloudera.org:8080/#/c/7759/2/src/kudu/tools/ksck.cc File src/kudu/tools/ksck.cc: Line 57: "If set, ksck prints only summary information. Use with the --format flag to " > Does this description still make sense when also doing a checksum check? In does, in the sense that the checksum output will also be suppressed (though still reflected in the return status/value of ksck). -- To view, visit http://gerrit.cloudera.org:8080/7759 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3a961867a9e6a32ca91144843f9d1b8139f756f5 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-HasComments: Yes
