(doris) branch master updated: [opt](nereids) optimize one bucket tpcds performance (#47371)

[morrysnow]

This is an automated email from the ASF dual-hosted git repository.

morrysnow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new b4dcf89aa93 [opt](nereids) optimize one bucket tpcds performance 
(#47371)
b4dcf89aa93 is described below

commit b4dcf89aa93e0692a73c6cf4321a623792f2dd3a
Author: 924060929 <lanhuaj...@selectdb.com>
AuthorDate: Thu Jan 23 21:43:01 2025 +0800

    [opt](nereids) optimize one bucket tpcds performance (#47371)
    
    ### What problem does this PR solve?
    
    use one bucket shuffle hash join is slow than shuffle hash join, so we
    should downgrade to shuffle hash join when the table only contains one
    bucket.
    
    this pr is optimized for nereids distribute planner in master branch
---
 .../apache/doris/nereids/properties/ChildrenPropertiesRegulator.java   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java
 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java
index b821ff0de87..ee1fd0b12ee 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java
@@ -218,8 +218,7 @@ public class ChildrenPropertiesRegulator extends 
PlanVisitor<List<List<PhysicalP
         boolean isEnableBucketShuffleJoin = 
ConnectContext.get().getSessionVariable().isEnableBucketShuffleJoin();
         if (!isEnableBucketShuffleJoin) {
             return true;
-        } else if (otherSideSpec.getShuffleType() != 
ShuffleType.EXECUTION_BUCKETED
-                || !(oneSidePlan instanceof GroupPlan)) {
+        } else if (!(oneSidePlan instanceof GroupPlan)) {
             return false;
         } else {
             PhysicalOlapScan candidate = 
findDownGradeBucketShuffleCandidate((GroupPlan) oneSidePlan);


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org