database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[minor](Nereids): remove useless stream filter() in Translator (#33758)

Browse Source
This commit is contained in:
jakevin
2024-04-17 18:35:48 +08:00
committed by yiguolei
parent ee3b6fdf58
commit 461561fed0
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java
View File

@ -1383,7 +1383,6 @@ public class PhysicalPlanTranslator extends DefaultPlanVisitor<PlanFragment, Pla
hashJoin.getOtherJoinConjuncts()
.stream()
.filter(e -> !(e.equals(BooleanLiteral.TRUE)))
.flatMap(e -> e.getInputSlots().stream())
.map(SlotReference.class::cast)
.forEach(s -> hashOutputSlotReferenceMap.put(s.getExprId(), s));
@ -1525,11 +1524,10 @@ public class PhysicalPlanTranslator extends DefaultPlanVisitor<PlanFragment, Pla
leftIntermediateSlotDescriptor.forEach(sd -> sd.setIsNullable(true));
}
// Constant expr will cause be crash.
// But EliminateJoinCondition and Expression Rewrite already eliminate true literal.
List<Expr> otherJoinConjuncts = hashJoin.getOtherJoinConjuncts()
.stream()
// TODO add constant expr will cause be crash, currently we only handle true literal.
// remove it after Nereids could ensure no constant expr in other join condition
.filter(e -> !(e.equals(BooleanLiteral.TRUE)))
.map(e -> ExpressionTranslator.translate(e, context))
.collect(Collectors.toList());