This is an automated email from the ASF dual-hosted git repository.

kxiao pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git

commit 0456342faadadd5cd038472d2c479040af14947f
Author: minghong <engle...@gmail.com>
AuthorDate: Fri Jul 14 11:30:25 2023 +0800

    [improve](nereids)inner join estimation: assume children output at least 
one tuple #21792
    
    this assumption is good to eliminate error propagation, when the filter 
estimation is too low, less than one.
---
 .../src/main/java/org/apache/doris/nereids/stats/JoinEstimation.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/JoinEstimation.java 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/JoinEstimation.java
index 228b6936a7..867ad1527c 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/JoinEstimation.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/stats/JoinEstimation.java
@@ -124,7 +124,7 @@ public class JoinEstimation {
 
         Statistics innerJoinStats;
         Statistics crossJoinStats = new StatisticsBuilder()
-                .setRowCount(Math.max(1, leftStats.getRowCount() * 
rightStats.getRowCount()))
+                .setRowCount(Math.max(1, leftStats.getRowCount()) * 
Math.max(1, rightStats.getRowCount()))
                 .putColumnStatistics(leftStats.columnStatistics())
                 .putColumnStatistics(rightStats.columnStatistics())
                 .build();


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to