Hello Yida Wu, Abhishek Rawat, Wenzhe Zhou, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20606
to look at the new patch set (#3).
Change subject: IMPALA-12510: Floor PlanFragment.maxParallelism_ at 1
......................................................................
IMPALA-12510: Floor PlanFragment.maxParallelism_ at 1
IMPALA-12444 introduce a bug where PlanFragment.maxParallelism_ can be
set to 0. This can happen at scan fragment if table is empty. Number of
scan ranges will be 0, which then propagate to
ScanNode.maxScannerThreads_ and PlanFragment.maxParallelism_.
This patch fix it by flooring ScanNode.maxScannerThreads_ and
PlanFragment.maxParallelism_ at 1.
Testing:
- Add select star over an empty table testcase to
PlannerTest.testProcessingCost.
Change-Id: Ibfa50abfdb9cdb994c5c3d7904b377a25f5b8b97
---
M fe/src/main/java/org/apache/impala/planner/PlanFragment.java
M fe/src/main/java/org/apache/impala/planner/ScanNode.java
M
testdata/workloads/functional-planner/queries/PlannerTest/tpcds-processing-cost.test
3 files changed, 46 insertions(+), 11 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/06/20606/3
--
To view, visit http://gerrit.cloudera.org:8080/20606
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ibfa50abfdb9cdb994c5c3d7904b377a25f5b8b97
Gerrit-Change-Number: 20606
Gerrit-PatchSet: 3
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Reviewer: Yida Wu <[email protected]>
