Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15259 )
Change subject: IMPALA-4080 [part 2]: Invoke codegen from scan's plan node instead of exec node ...................................................................... IMPALA-4080 [part 2]: Invoke codegen from scan's plan node instead of exec node This patch moves the code responsible for invoking codegen for hdfs scanner to the plan node. Testing: Successfully ran exhaustive tests. Also verified manually that codegen code is generated and used for each hdfs scanner. Change-Id: I02c6183271b3731aa0d3cae2426be7269e462967 Reviewed-on: http://gerrit.cloudera.org:8080/15259 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/exec/hdfs-avro-scanner-ir.cc M be/src/exec/hdfs-avro-scanner.cc M be/src/exec/hdfs-avro-scanner.h M be/src/exec/hdfs-orc-scanner.cc M be/src/exec/hdfs-rcfile-scanner.cc M be/src/exec/hdfs-rcfile-scanner.h 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-scanner.h M be/src/exec/hdfs-sequence-scanner.cc M be/src/exec/hdfs-sequence-scanner.h M be/src/exec/hdfs-text-scanner.cc M be/src/exec/hdfs-text-scanner.h M be/src/exec/parquet/hdfs-parquet-scanner.cc M be/src/exec/parquet/hdfs-parquet-scanner.h 16 files changed, 294 insertions(+), 263 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/15259 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I02c6183271b3731aa0d3cae2426be7269e462967 Gerrit-Change-Number: 15259 Gerrit-PatchSet: 5 Gerrit-Owner: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
