fix

englefly · Sep 22, 2023 · a588f8f · a588f8f
1 parent dd8a304
commit a588f8f
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/FilterEstimation.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/FilterEstimation.java
@@ -393,11 +393,14 @@ public Statistics visitIsNull(IsNull isNull, EstimationContext context) {
         if (childStats.isUnKnown()) {
             return new StatisticsBuilder(context.statistics).build();
         }
-        double outputRowCount = context.statistics.getRowCount() - childStats.numNulls;
+        double outputRowCount = childStats.numNulls;
+        ColumnStatisticBuilder colBuilder = new ColumnStatisticBuilder(childStats);
+        colBuilder.setCount(outputRowCount).setNumNulls(outputRowCount).setNdv(0)
+                .setMinValue(Double.NEGATIVE_INFINITY).setMaxValue(Double.POSITIVE_INFINITY);
         StatisticsBuilder builder = new StatisticsBuilder(context.statistics);
-        builder.putColumnStatistics(isNull.child(), childStats);
-        Statistics outputStats = builder.build().updateRowCountOnly(outputRowCount);
-        return outputStats;
+        builder.putColumnStatistics(isNull.child(), colBuilder.build());
+        // TODO we do not call updateRowCountOnly() to make is-not-null work. this need refactor
+        return builder.build();
     }
 
     static class EstimationContext {