[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: Code-Review+2 Given that block_manager-stress-test is

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. KUDU-2665: deflake block_manager-stress-test After commit

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: Verified+1 Overriding Jenkins; the Python3 build breakages

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: > Reread the ThreadPool code, it seems that

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: Code-Review+1 Reread the ThreadPool code, it seems that

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: my preference 2) is not safe because we can't be sure when the

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread Adar Dembo (Code Review)
Hello Mike Percy, helifu, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12254 to review the following change. Change subject: KUDU-2665: deflake block_manager-stress-test .. KUDU-2665: