fix feut

feiniaofeiafei · Jan 3, 2025 · 6614e8c · 6614e8c
1 parent d419299
commit 6614e8c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalRepeat.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalRepeat.java
@@ -202,6 +202,8 @@ public void computeUniform(DataTrait.Builder builder) {
             Optional<Expression> value = entry.getValue();
             if (!value.isPresent() || value.get() instanceof NullLiteral || common.contains(value.get())) {
                 newSlotUniformValue.put(entry.getKey(), value);
+            } else {
+                newSlotUniformValue.put(entry.getKey(), Optional.empty());
             }
         }
         builder.addUniformSlotValueMap(newSlotUniformValue);

diff --git a/fe/fe-core/src/test/java/org/apache/doris/nereids/properties/DataTraitTest.java b/fe/fe-core/src/test/java/org/apache/doris/nereids/properties/DataTraitTest.java
@@ -242,8 +242,8 @@ void testAgg() {
                 .rewrite()
                 .getPlan();
         System.out.println(plan.getLogicalProperties().getTrait());
-        Assertions.assertTrue(plan.getLogicalProperties()
-                .getTrait().isUniform(plan.getOutput().get(0)));
+        Assertions.assertTrue(plan.getLogicalProperties().getTrait().isUniform(plan.getOutput().get(0))
+                && !plan.getLogicalProperties().getTrait().isUniformAndNotNull(plan.getOutput().get(0)));
     }
 
     @Test