Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16567 )
Change subject: IMPALA-9485: Enable file handle cache for EC files ...................................................................... IMPALA-9485: Enable file handle cache for EC files This is essentially a revert of IMPALA-8178. HDFS-14308 added CanUnbuffer support to the EC input stream APIs in the HDFS client lib. This patch enables file handle caching for EC files. Testing: * Ran core tests against an EC build (ERASURE_CODING=true) Change-Id: Ieb455eeed02a229a4559d3972dfdac7df32cdb99 Reviewed-on: http://gerrit.cloudera.org:8080/16567 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/exec/base-sequence-scanner.cc M be/src/exec/hdfs-orc-scanner.cc M be/src/exec/hdfs-scan-node-base.cc M be/src/exec/hdfs-scan-node-base.h M be/src/exec/hdfs-scanner.cc M be/src/exec/hdfs-text-scanner.cc M be/src/exec/parquet/hdfs-parquet-scanner.cc M be/src/exec/parquet/parquet-page-index.cc M be/src/exec/parquet/parquet-page-reader.cc M be/src/exec/scanner-context.cc M be/src/runtime/io/disk-io-mgr-stress.cc M be/src/runtime/io/disk-io-mgr-test.cc M be/src/runtime/io/request-ranges.h M be/src/runtime/io/scan-range.cc M be/src/runtime/tmp-file-mgr.cc M be/src/scheduling/scheduler.cc M common/thrift/PlanNodes.thrift M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java M tests/custom_cluster/test_hdfs_fd_caching.py 19 files changed, 50 insertions(+), 82 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16567 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ieb455eeed02a229a4559d3972dfdac7df32cdb99 Gerrit-Change-Number: 16567 Gerrit-PatchSet: 4 Gerrit-Owner: Sahil Takiar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]>
