[opt](nereids) refine operator estimation

xzj7019 · Sep 19, 2024 · dd61c8f · dd61c8f
1 parent 3842d9f
commit dd61c8f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/ExpressionEstimation.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/ExpressionEstimation.java
@@ -244,7 +244,7 @@ public ColumnStatistic visitLiteral(Literal literal, Statistics context) {
                 .setMinValue(literalVal)
                 .setNdv(literal.isNullLiteral() ? 0 : 1)
                 .setNumNulls(literal.isNullLiteral() ? 1 : 0)
-                .setAvgSizeByte(1)
+                .setAvgSizeByte(literal.getDataType().width())
                 .setMinExpr(literal.toLegacyLiteral())
                 .setMaxExpr(literal.toLegacyLiteral())
                 .build();

diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/StatsCalculator.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/StatsCalculator.java
@@ -406,7 +406,7 @@ private Statistics computeOlapScan(OlapScan olapScan) {
                             derivedStats.addColumnStats(slot,
                                     new ColumnStatisticBuilder(ColumnStatistic.UNKNOWN)
                                             .setCount(derivedRowCount)
-                                            .setAvgSizeByte(slot.getDataType().width())
+                                            //.setAvgSizeByte(slot.getDataType().width())
                                             .build());
                         }
                     }
@@ -436,7 +436,7 @@ private Statistics computeOlapScan(OlapScan olapScan) {
                 builder.putColumnStatistics(slot,
                         new ColumnStatisticBuilder(ColumnStatistic.UNKNOWN)
                                 .setCount(tableRowCount)
-                                .setAvgSizeByte(slot.getDataType().width())
+                                //.setAvgSizeByte(slot.getDataType().width())
                                 .build());
             }
             setHasUnknownColStatsInStatementContext();