This is an automated email from the ASF dual-hosted git repository.
englefly pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new ea73dd5851 [improve](nereids)inner join estimation: assume children
output at least one tuple #21792
ea73dd5851 is described below
commit ea73dd585178fd93c978510145baf190e26f6511
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 afef2285e1..36dc90b343 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
@@ -115,7 +115,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
