Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20606 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/20606 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Riza Suminto <riza.sumi...@cloudera.com> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved Riza Suminto: Verified -- 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: merged Gerrit-Change-Id: Ibfa50abfdb9cdb994c5c3d7904b377a25f5b8b97 Gerrit-Change-Number: 20606 Gerrit-PatchSet: 5 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com> Gerrit-Reviewer: Yida Wu <wydbaggio...@gmail.com>
