924060929 commented on code in PR #67177:
URL: https://github.com/apache/doris/pull/67177#discussion_r3870549326
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/distribute/DistributePlanner.java:
##########
@@ -75,19 +78,37 @@ public class DistributePlanner {
private final boolean notNeedBackend;
private final boolean isLoadJob;
private final boolean useLoadBackendSelection;
+ private final int beExecVersion;
public DistributePlanner(StatementContext statementContext,
List<PlanFragment> fragments, boolean notNeedBackend, boolean
isLoadJob,
boolean useLoadBackendSelection) {
+ this(statementContext, fragments, notNeedBackend, isLoadJob,
useLoadBackendSelection,
+ Config.be_exec_version);
+ }
+
+ public DistributePlanner(StatementContext statementContext,
Review Comment:
Addressed by the final refactor. The added public `DistributePlanner`
constructor was removed entirely; local-exchange enforcement now stays in
`PlanNode.enforceRequire`, so there is no new constructor or Javadoc
requirement in the current diff.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
