Joe McDonnell has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/6312

Change subject: IMPALA-4996: Single-threaded KuduScanNode
......................................................................

IMPALA-4996: Single-threaded KuduScanNode

This introduces KuduScanNodeMt, the single-threaded version of KuduScanNode
that materializes the tuples in GetNext(). KuduScanNodeMt is enabled by the
same condition as HdfsScanNodeMt: mt_dop is greater than 1.

To share code between the two implementations, KuduScanNode and KuduScanNodeMt
are now subclasses of KuduScanNodeBase, which implements the shared code.
The KuduScanner is minimally impacted, as it already had the required
GetNext interface.

The testing for this is a modified version of kudu-scan-node.test run with
various mt_dop values.

Change-Id: I6e4593300e376bc508b78acaea64ffdd2c73a67a
---
M be/src/exec/CMakeLists.txt
M be/src/exec/exec-node.cc
M be/src/exec/hdfs-scan-node-base.h
A be/src/exec/kudu-scan-node-base.cc
A be/src/exec/kudu-scan-node-base.h
A be/src/exec/kudu-scan-node-mt.cc
A be/src/exec/kudu-scan-node-mt.h
M be/src/exec/kudu-scan-node.cc
M be/src/exec/kudu-scan-node.h
M be/src/exec/kudu-scanner.cc
M be/src/exec/kudu-scanner.h
M common/thrift/PlanNodes.thrift
M fe/src/main/java/org/apache/impala/planner/KuduScanNode.java
A testdata/workloads/functional-query/queries/QueryTest/mt-dop-kudu.test
M tests/query_test/test_mt_dop.py
15 files changed, 515 insertions(+), 137 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6e4593300e376bc508b78acaea64ffdd2c73a67a
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Joe McDonnell <[email protected]>

Reply via email to