[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-24 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 14: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 14: Code-Review+2 Leaving this open in case Alexey or Will

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-24 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#14). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-24 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#13). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 13: (2 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-23 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 12: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-23 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 12: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-22 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 12: (2 comments) Just the two nits and LGTM.

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 12: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 12: (3 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#12). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 11: (3 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 11: Code-Review+1 The test failures all seem to be clock sync

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 11: (2 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#11). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 10: (2 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 10: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#10). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 9: (2 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#9). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-15 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 8: (4 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-15 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#8). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-15 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 7: (2 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-15 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#7). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-14 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 6: (2 comments) Couple nits, then SGTM.

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-12 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 6: > Patch Set 6: Verified-1 > > Build Failed > >

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-12 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-12 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 5: (6 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-12 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#5). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 4: (4 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 2: (5 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-10 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 4: (1 comment) > Patch Set 2: > > (1 comment) thanks for

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-10 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#4). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-10 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#3). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-09 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 2: (1 comment) > Patch Set 2: > > (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 2: Yea, I dont think adding a new Status is the right

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-03 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 2: I wasn't sure adding a new Status for this is the best

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-03 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#2). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-03 Thread Attila Bukor (Code Review)
Attila Bukor has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10293 Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. KUDU-2426 Fix WRONG_SERVER_UUID case in ksck