[Impala-ASF-CR](2.x) IMPALA-6549: Enable file handle cache by default
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9426 ) Change subject: IMPALA-6549: Enable file handle cache by default .. Patch Set 2: Verified+1 Code-Review+2 Tested with https://gerrit.cloudera.org/#/c/9429/ -- To view, visit http://gerrit.cloudera.org:8080/9426 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: comment Gerrit-Change-Id: I6935825a1c4c7b2da0bb877f732027be1a57a8b7 Gerrit-Change-Number: 9426 Gerrit-PatchSet: 2 Gerrit-Owner: Tim ArmstrongGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Joe McDonnell Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Sat, 24 Feb 2018 01:58:39 + Gerrit-HasComments: No
[Impala-ASF-CR](2.x) IMPALA-6549: Enable file handle cache by default
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9426 ) Change subject: IMPALA-6549: Enable file handle cache by default .. IMPALA-6549: Enable file handle cache by default The file handle cache was disabled by default due to two HDFS issues: HDFS-12528 and HDFS-14872. Both have been fixed and the CDH components in the toolchain include both fixes. This reenables the file handle cache by default. Change-Id: I6935825a1c4c7b2da0bb877f732027be1a57a8b7 Reviewed-on: http://gerrit.cloudera.org:8080/9371 Reviewed-by: Joe McDonnellTested-by: Impala Public Jenkins Reviewed-on: http://gerrit.cloudera.org:8080/9426 Reviewed-by: Tim Armstrong Tested-by: Tim Armstrong --- M be/src/runtime/io/disk-io-mgr.cc 1 file changed, 7 insertions(+), 9 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/9426 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: merged Gerrit-Change-Id: I6935825a1c4c7b2da0bb877f732027be1a57a8b7 Gerrit-Change-Number: 9426 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Joe McDonnell Gerrit-Reviewer: Tim Armstrong
[Impala-ASF-CR](2.x) IMPALA-6549: Enable file handle cache by default
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/9426 ) Change subject: IMPALA-6549: Enable file handle cache by default .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9426 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: comment Gerrit-Change-Id: I6935825a1c4c7b2da0bb877f732027be1a57a8b7 Gerrit-Change-Number: 9426 Gerrit-PatchSet: 1 Gerrit-Owner: Tim ArmstrongGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Joe McDonnell Gerrit-Comment-Date: Fri, 23 Feb 2018 17:41:48 + Gerrit-HasComments: No
[Impala-ASF-CR](2.x) IMPALA-6549: Enable file handle cache by default
Hello Joe McDonnell, Impala Public Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9426 to review the following change. Change subject: IMPALA-6549: Enable file handle cache by default .. IMPALA-6549: Enable file handle cache by default The file handle cache was disabled by default due to two HDFS issues: HDFS-12528 and HDFS-14872. Both have been fixed and the CDH components in the toolchain include both fixes. This reenables the file handle cache by default. Change-Id: I6935825a1c4c7b2da0bb877f732027be1a57a8b7 Reviewed-on: http://gerrit.cloudera.org:8080/9371 Reviewed-by: Joe McDonnellTested-by: Impala Public Jenkins --- M be/src/runtime/io/disk-io-mgr.cc 1 file changed, 7 insertions(+), 9 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/26/9426/1 -- To view, visit http://gerrit.cloudera.org:8080/9426 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: newchange Gerrit-Change-Id: I6935825a1c4c7b2da0bb877f732027be1a57a8b7 Gerrit-Change-Number: 9426 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Joe McDonnell