Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9912 )
Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 ...................................................................... [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 This improves formatting in ksck output and ensures that the tablet ids appear in the output in verbose mode (KUDU-2271). It also revamps how results are printed when tablet id filters are specified, omitting conclusions about entire tables. It's now clear from ksck's messages whether it is checking individual tablets using tablet filters or checking whole tables. This addresses KUDU-2310 as well, since it makes it clear what sort of filters are being applied. The summary table is unchanged since it's nice whether a table or tablet filter is applied. Change-Id: I2371b704a7c2ebb1b5a4822e3b4d852443c200c5 Reviewed-on: http://gerrit.cloudera.org:8080/9912 Reviewed-by: Attila Bukor <[email protected]> Tested-by: Will Berkeley <[email protected]> Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck.cc M src/kudu/tools/ksck.h 3 files changed, 107 insertions(+), 20 deletions(-) Approvals: Attila Bukor: Looks good to me, but someone else must approve Will Berkeley: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9912 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2371b704a7c2ebb1b5a4822e3b4d852443c200c5 Gerrit-Change-Number: 9912 Gerrit-PatchSet: 6 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
