[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. [tools] ksck improvements [7/n] Add JSON

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 12: Verified+1 -- To view, visit

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Will Berkeley (Code Review)
Will Berkeley has removed a vote on this change. Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10288

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 12: Code-Review+2 -- To view, visit

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 12: Code-Review+2 -- To view, visit

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 11: (10 comments) Note also that the

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#12). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 11: (10 comments) Overall LGTM, just a

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-11 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#11). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 10: Code-Review+2 -- To view, visit

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-10 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#10). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 9: (9 comments)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 9: (9 comments) Mostly nits, looks good!

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-10 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#9). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 8: (4 comments)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 8: (4 comments)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-05 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#8). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-05 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 7: (9 comments)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 3: (10 comments)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#7). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#6). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 3: (3 comments) Since we're bikeshedding

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#5). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 3: (3 comments) Thank you for getting

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#4). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 1: (38 comments)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#3). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10288 to look at the new patch set (#2). Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-02 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 1: (1 comment)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 1: (1 comment)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-02 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 1: (1 comment)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10288 ) Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. Patch Set 1: (1 comment)

[kudu-CR] [tools] ksck improvements [7/n] Add JSON output option to ksck

2018-05-02 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10288 Change subject: [tools] ksck improvements [7/n] Add JSON output option to ksck .. [tools] ksck improvements [7/n] Add JSON output