You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Jim Apple (Code Review)" <ge...@cloudera.org> on 2016/05/19 21:07:33 UTC

[Impala-CR](cdh5-trunk) IMPALA-3450: LIMITs on plan nodes are reflected in cardinality estimates

Jim Apple has uploaded a new patch set (#4).

Change subject: IMPALA-3450: LIMITs on plan nodes are reflected in cardinality estimates
......................................................................

IMPALA-3450: LIMITs on plan nodes are reflected in cardinality estimates

PlanNode includes a 'capAtLimit()' method that can be used in
'computeStats()' on PlanNodes to ensure they do not estimate their
cardinality to be more than a pushed-down LIMIT clause.

This patch ensures that 'capAtLimit()' is used in all of the relevant
classes descending from PlanNode.

Change-Id: Ic06dcb93bbb2510c0d40151302bd817ef340b825
---
M fe/src/main/java/com/cloudera/impala/planner/AggregationNode.java
M fe/src/main/java/com/cloudera/impala/planner/JoinNode.java
M fe/src/main/java/com/cloudera/impala/planner/SubplanNode.java
M fe/src/main/java/com/cloudera/impala/planner/UnionNode.java
M fe/src/main/java/com/cloudera/impala/planner/UnnestNode.java
M testdata/workloads/functional-planner/queries/PlannerTest/aggregation.test
M testdata/workloads/functional-planner/queries/PlannerTest/join-order.test
M testdata/workloads/functional-planner/queries/PlannerTest/joins.test
M testdata/workloads/functional-planner/queries/PlannerTest/nested-collections.test
M testdata/workloads/functional-planner/queries/PlannerTest/union.test
10 files changed, 346 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/27/3127/4
-- 
To view, visit http://gerrit.cloudera.org:8080/3127
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic06dcb93bbb2510c0d40151302bd817ef340b825
Gerrit-PatchSet: 4
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Jim Apple <jb...@cloudera.com>
Gerrit-Reviewer: Alex Behm <al...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jb...@cloudera.com>