zhengshiJ commented on code in PR #8947:
URL: https://github.com/apache/incubator-doris/pull/8947#discussion_r853716965
##########
fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java:
##########
@@ -175,26 +176,32 @@ protected PlanNode(PlanNodeId id, PlanNode node, String
planNodeName) {
"V" + planNodeName : planNodeName;
this.numInstances = 1;
this.nodeType = node.getNodeType();
- this.statsDeriveResult.set(node.getStatsDeriveResult());
+ this.statsDeriveResult = new StatsDeriveResult(
Review Comment:
When the derivation of statistics for all nodes is complete, the
initialization needs to be removed.
The reason for the current increase is that some uts cannot pass
##########
fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java:
##########
@@ -133,7 +134,7 @@ abstract public class PlanNode extends TreeNode<PlanNode> {
protected List<SlotId> outputSlotIds;
- private NodeType nodeType = NodeType.DEFAULT;
+ protected NodeType nodeType = NodeType.DEFAULT;
protected StatsDeriveResult statsDeriveResult = new StatsDeriveResult();
Review Comment:
When the derivation of statistics for all nodes is complete, the
initialization needs to be removed.
The reason for the current increase is that some uts cannot pass
--
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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
