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

[Feature](materialized-view) support rewrite case when to if on legacy planner to make mv work (#30320)

Browse Source 
support rewrite case when to if on legacy planner to make mv work
This commit is contained in:
Pxl
2024-01-25 10:06:37 +08:00
committed by yiguolei
parent 5ba9987d3a
commit 5b462194d1
5 changed files with 113 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
fe/fe-core/src/main/java/org/apache/doris/analysis/Analyzer.java
View File

@ -48,6 +48,7 @@ import org.apache.doris.planner.RuntimeFilter;
import org.apache.doris.qe.ConnectContext;
import org.apache.doris.qe.SessionVariable;
import org.apache.doris.rewrite.BetweenToCompoundRule;
import org.apache.doris.rewrite.CaseWhenToIf;
import org.apache.doris.rewrite.CompoundPredicateWriteRule;
import org.apache.doris.rewrite.ElementAtToSlotRefRule;
import org.apache.doris.rewrite.EliminateUnnecessaryFunctions;
@ -460,6 +461,7 @@ public class Analyzer {
rules.add(EliminateUnnecessaryFunctions.INSTANCE);
rules.add(ElementAtToSlotRefRule.INSTANCE);
rules.add(FunctionAlias.INSTANCE);
rules.add(CaseWhenToIf.INSTANCE);
List<ExprRewriteRule> onceRules = Lists.newArrayList();
onceRules.add(ExtractCommonFactorsRule.INSTANCE);
onceRules.add(InferFiltersRule.INSTANCE);

14
fe/fe-core/src/main/java/org/apache/doris/analysis/CaseExpr.java
View File

@ -352,11 +352,7 @@ public class CaseExpr extends Expr {
}
if (caseExpr instanceof NullLiteral) {
if (expr.hasElseExpr) {
return expr.getChild(expr.getChildren().size() - 1);
} else {
return new NullLiteral();
}
return expr.getFinalResult();
}
if (expr.hasElseExpr) {
@ -402,8 +398,12 @@ public class CaseExpr extends Expr {
}
}
if (expr.hasElseExpr) {
return expr.getChild(expr.getChildren().size() - 1);
return expr.getFinalResult();
}
public Expr getFinalResult() {
if (hasElseExpr) {
return getChild(getChildren().size() - 1);
} else {
return new NullLiteral();
}

52
fe/fe-core/src/main/java/org/apache/doris/rewrite/CaseWhenToIf.java Normal file
View File

@ -0,0 +1,52 @@
// Licensed to the Apache Software Foundation (ASF) under one
// or more contributor license agreements. See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership. The ASF licenses this file
// to you under the Apache License, Version 2.0 (the
// "License"); you may not use this file except in compliance
// with the License. You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing,
// software distributed under the License is distributed on an
// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
package org.apache.doris.rewrite;
import org.apache.doris.analysis.Analyzer;
import org.apache.doris.analysis.CaseExpr;
import org.apache.doris.analysis.Expr;
import org.apache.doris.analysis.FunctionCallExpr;
import org.apache.doris.common.AnalysisException;
import com.google.common.collect.Lists;
import java.util.ArrayList;
/**
* Change function 'case when' to 'if', same with
* nereids/rules/expression/rules/CaseWhenToIf.java
*/
public final class CaseWhenToIf implements ExprRewriteRule {
public static ExprRewriteRule INSTANCE = new CaseWhenToIf();
@Override
public Expr apply(Expr expr, Analyzer analyzer, ExprRewriter.ClauseType clauseType) throws AnalysisException {
if (!(expr instanceof CaseExpr)) {
return expr;
}
CaseExpr caseWhen = (CaseExpr) expr;
if (caseWhen.getConditionExprs().size() == 1) {
ArrayList<Expr> ifArgs = Lists.newArrayList();
ifArgs.add(caseWhen.getConditionExprs().get(0));
ifArgs.add(caseWhen.getReturnExprs().get(0));
ifArgs.add(caseWhen.getFinalResult());
return new FunctionCallExpr("if", ifArgs);
}
return expr;
}
}