Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16880 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/16880 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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, 86 insertions(+), 18 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I72e341597e980fb6a7e3792905b942ddf5797d03 Gerrit-Change-Number: 16880 Gerrit-PatchSet: 5 Gerrit-Owner: Akos Kovacs <[email protected]> Gerrit-Reviewer: Akos Kovacs <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
