Riza Suminto has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21021


Change subject: IMPALA-12796: Add is_footer_only in TFileSplitGeneratorSpec
......................................................................

IMPALA-12796: Add is_footer_only in TFileSplitGeneratorSpec

Several tests in test_scanners.py failed by wrong row counts with S3
target filesystem after IMPALA-12631. S3 filesystem does not have block.
Planner will produce TFileSplitGeneratorSpec instead of
TScanRangeLocationList, and IMPALA-12631 miss to address necessary
changes in TFileSplitGeneratorSpec.

This patch address the issue by adding is_footer_only field in
TFileSplitGeneratorSpec schedule accordingly in schedule.cc.

Testing:
- Pass core tests with S3 target filesystem.

Change-Id: Iaa6e3c14debe68cf601131c6594774c8c695923e
---
M be/src/scheduling/scheduler-test-util.cc
M be/src/scheduling/scheduler-test-util.h
M be/src/scheduling/scheduler-test.cc
M be/src/scheduling/scheduler.cc
M common/thrift/PlanNodes.thrift
M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java
6 files changed, 67 insertions(+), 25 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/21/21021/1
--
To view, visit http://gerrit.cloudera.org:8080/21021
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaa6e3c14debe68cf601131c6594774c8c695923e
Gerrit-Change-Number: 21021
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>

Reply via email to