This is an automated email from the ASF dual-hosted git repository.
yiguolei 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 0c5b4ecc7c [fix](agg)repeat node shouldn't change slot's nullable
property of agg node (#11859)
0c5b4ecc7c is described below
commit 0c5b4ecc7cc46e2b014bb1f4c909be04f736c54a
Author: starocean999 <40539150+starocean...@users.noreply.github.com>
AuthorDate: Mon Aug 22 16:28:45 2022 +0800
[fix](agg)repeat node shouldn't change slot's nullable property of agg node
(#11859)
---
fe/fe-core/src/main/java/org/apache/doris/planner/RepeatNode.java | 8 --------
1 file changed, 8 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/planner/RepeatNode.java
b/fe/fe-core/src/main/java/org/apache/doris/planner/RepeatNode.java
index 5fef371ec8..f9f136f208 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/planner/RepeatNode.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/planner/RepeatNode.java
@@ -24,10 +24,8 @@ import org.apache.doris.analysis.GroupByClause;
import org.apache.doris.analysis.GroupingInfo;
import org.apache.doris.analysis.SlotDescriptor;
import org.apache.doris.analysis.SlotId;
-import org.apache.doris.analysis.SlotRef;
import org.apache.doris.analysis.TupleDescriptor;
import org.apache.doris.analysis.TupleId;
-import org.apache.doris.analysis.VirtualSlotRef;
import org.apache.doris.common.UserException;
import org.apache.doris.statistics.StatisticalType;
import org.apache.doris.statistics.StatsRecursiveDerive;
@@ -126,12 +124,6 @@ public class RepeatNode extends PlanNode {
outputTupleDesc.setTable(analyzer.getTupleDesc(inputTupleIds.get(0)).getTable());
}
- //set aggregate nullable
- for (Expr slot : groupByClause.getGroupingExprs()) {
- if (slot instanceof SlotRef && !(slot instanceof VirtualSlotRef)) {
- ((SlotRef) slot).getDesc().setIsNullable(true);
- }
- }
outputTupleDesc.computeStatAndMemLayout();
List<Set<SlotId>> groupingIdList = new ArrayList<>();
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
