[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.o

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. IMPALA-6953: part 2: clean up DiskIoMgr Follow-on changes.

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-ve

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-ve

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 3: Build started: https://jenkins.impala.io/job/gerrit-ve

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-ve

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.clouder

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 3: Code-Review+2 Fix a clang-tidy error. Need to investigate the

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-23 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10479 to look at the new patch set (#3). Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. I

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 2: Verified-1 Build failed: https://jenkins.impala.io/job

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 2: Code-Review+2 (2 comments) carry http://gerrit.cloudera.org:

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 2: Build started: https://jenkins.impala.io/job/gerrit-ve

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-22 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10479 to look at the new patch set (#2). Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. IMPALA-6953: part 2: cle

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10479 ) Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. Patch Set 1: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/10

[Impala-ASF-CR] IMPALA-6953: part 2: clean up DiskIoMgr

2018-05-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10479 Change subject: IMPALA-6953: part 2: clean up DiskIoMgr .. IMPALA-6953: part 2: clean up DiskIoMgr Follow-on changes. I separated this