database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[fix](planner) Fix an issue where outputSmap's size could grow exponentially (#11378)

Browse Source 
* [fix](planner) Fix an issue where outputSmap'size  could grow exponentially
This commit is contained in:
Mingyu Chen
2022-08-03 09:07:00 +08:00
committed by GitHub
parent 71ea3f2906
commit db3ba02993
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
fe/fe-core/src/main/java/org/apache/doris/analysis/ExprSubstitutionMap.java
View File

@ -25,8 +25,8 @@ import org.apache.doris.common.AnalysisException;
import com.google.common.base.Joiner;
import com.google.common.base.Preconditions;
import com.google.common.collect.Lists;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import java.util.List;
@ -39,7 +39,7 @@ import java.util.List;
* See Expr.substitute() and related functions for details on the actual substitution.
*/
public final class ExprSubstitutionMap {
private static final Logger LOG = LoggerFactory.getLogger(ExprSubstitutionMap.class);
private static final Logger LOG = LogManager.getLogger(ExprSubstitutionMap.class);
private boolean checkAnalyzed = true;
private List<Expr> lhs; // left-hand side
@ -209,7 +209,9 @@ public final class ExprSubstitutionMap {
Expr fRhs = f.getRhs().get(j);
if (fRhs.contains(gLhs)) {
Expr newRhs = fRhs.trySubstitute(g, analyzer, false);
result.put(f.getLhs().get(j), newRhs);
if (!result.containsMappingFor(f.getLhs().get(j))) {
result.put(f.getLhs().get(j), newRhs);
}
findGMatch = true;
}
}