[kudu-CR] log block manager: corruptor test utility

2017-04-27 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: log block manager: corruptor test utility .. log block manager: corruptor test utility This patch introduces the LBMCorruptor, a test-only class for adding

[kudu-CR] log block manager: corruptor test utility

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http

[kudu-CR] log block manager: corruptor test utility

2017-04-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 11: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http://gerri

[kudu-CR] log block manager: corruptor test utility

2017-04-25 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6582 to look at the new patch set (#10). Change subject: log block manager: corruptor test utility .

[kudu-CR] log block manager: corruptor test utility

2017-04-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http://gerr

[kudu-CR] log block manager: corruptor test utility

2017-04-24 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6582 to look at the new patch set (#9). Change subject: log block manager: corruptor test utility ..

[kudu-CR] log block manager: corruptor test utility

2017-04-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6582/8/src/kudu/fs/log_block_manager-test-util.cc File sr

[kudu-CR] log block manager: corruptor test utility

2017-04-24 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6582 to look at the new patch set (#8). Change subject: log block manager: corruptor test utility ..

[kudu-CR] log block manager: corruptor test utility

2017-04-21 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/6582/7/src/kudu/fs/log_block_manager-test-util.cc File s

[kudu-CR] log block manager: corruptor test utility

2017-04-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/6582/7/src/kudu/fs/log_block_manager-test-util.cc Fil

[kudu-CR] log block manager: corruptor test utility

2017-04-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http:

[kudu-CR] log block manager: corruptor test utility

2017-04-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 7: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http://gerrit

[kudu-CR] log block manager: corruptor test utility

2017-04-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 7: Verified+1 Unrelated failure for which I pushed http://gerrit.cloudera.org:8080/6691. -- To view, visit

[kudu-CR] log block manager: corruptor test utility

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http:

[kudu-CR] log block manager: corruptor test utility

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http:

[kudu-CR] log block manager: corruptor test utility

2017-04-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 5: Verified+1 Unrelated test failure. -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe

[kudu-CR] log block manager: corruptor test utility

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit http:

[kudu-CR] log block manager: corruptor test utility

2017-04-10 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6582 to look at the new patch set (#4). Change subject: log block manager: corruptor test utility .. log block manager:

[kudu-CR] log block manager: corruptor test utility

2017-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 3: (17 comments) http://gerrit.cloudera.org:8080/#/c/6582/3/src/kudu/fs/block_manager-stress-test.cc File sr

[kudu-CR] log block manager: corruptor test utility

2017-04-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 3: (17 comments) http://gerrit.cloudera.org:8080/#/c/6582/3/src/kudu/fs/block_manager-stress-test.c

[kudu-CR] log block manager: corruptor test utility

2017-04-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6582 to look at the new patch set (#3). Change subject: log block manager: corruptor test utility .. log block manager:

[kudu-CR] log block manager: corruptor test utility

2017-04-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6582 to look at the new patch set (#2). Change subject: log block manager: corruptor test utility .. log block manager:

[kudu-CR] log block manager: corruptor test utility

2017-04-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/6582/1/src/kudu/fs/log_block_manager-test-util.cc File s

[kudu-CR] log block manager: corruptor test utility

2017-04-06 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6582 to review the following change. Change subject: log block manager: corruptor test utility .. log