You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by hu...@apache.org on 2026/03/31 06:39:43 UTC

(doris) 07/10: [fix](local shuffle) remove stale code in deriveAndEnforceChildLocalExchange

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

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

commit 2bac91f1a623f7b449719aebaab2a9c3561f8eb0
Author: 924060929 <la...@selectdb.com>
AuthorDate: Mon Mar 30 18:48:01 2026 +0800

    [fix](local shuffle) remove stale code in deriveAndEnforceChildLocalExchange
    
    Fix checkstyle parse error caused by leftover closing brace and unreachable
    return statement.
---
 fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java | 2 --
 1 file changed, 2 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java b/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java
index e5114f9cbd0..1bce63f1416 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java
@@ -1015,8 +1015,6 @@ public abstract class PlanNode extends TreeNode<PlanNode> {
                 : hasSerialAncestorInPipeline || isSerialOperator();
         translatorContext.setHasSerialAncestorInPipeline(child, childHasSerialAncestorInPipeline);
         return child.enforceAndDeriveLocalExchange(translatorContext, this, requireChild);
-        }
-        return result;
     }
 
     /**


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