You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Akos Kovacs (Code Review)" <ge...@cloudera.org> on 2021/01/05 11:31:00 UTC

[Impala-ASF-CR] IMPALA-9687 Improve estimates for number of hosts in Kudu plans

Hello Tim Armstrong, Impala Public Jenkins, 

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#2).

Change subject: IMPALA-9687 Improve estimates for number of hosts in Kudu plans
......................................................................

IMPALA-9687 Improve estimates for number of hosts in Kudu plans

In some cases Kudu plans could contain more hosts than the actual number of executors.
This commit fixes it by capping the number of hosts at the number of executors,
and determining which executors have local scan ranges.

Testing:
 - Ran core tests

Updated Kudu planner tests where the memory estimates changed.

Change-Id: I72e341597e980fb6a7e3792905b942ddf5797d03
---
M fe/src/main/java/org/apache/impala/planner/KuduScanNode.java
M testdata/workloads/functional-planner/queries/PlannerTest/kudu-selectivity.test
M testdata/workloads/functional-planner/queries/PlannerTest/tpch-kudu.test
3 files changed, 84 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/80/16880/2
-- 
To view, visit http://gerrit.cloudera.org:8080/16880
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I72e341597e980fb6a7e3792905b942ddf5797d03
Gerrit-Change-Number: 16880
Gerrit-PatchSet: 2
Gerrit-Owner: Akos Kovacs <ak...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>