fix bugs

Signed-off-by: shuming.li <[email protected]>
StarRocks · Aug 16, 2024 · eef41be · eef41be
1 parent a17b580
commit eef41be
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/fe/fe-core/src/main/java/com/starrocks/sql/optimizer/Optimizer.java b/fe/fe-core/src/main/java/com/starrocks/sql/optimizer/Optimizer.java
@@ -165,18 +165,14 @@ public OptExpression optimize(ConnectContext connectContext,
             // prepare for optimizer
             prepare(connectContext, columnRefFactory, logicOperatorTree);
 
+            // prepare for mv rewrite
+            prepareMvRewrite(connectContext, logicOperatorTree, columnRefFactory, requiredColumns);
             // try text based mv rewrite first before mv rewrite prepare so can deduce mv prepare time if it can be rewritten.
             try (Timer ignored = Tracers.watchScope("MVTextRewrite")) {
                 logicOperatorTree = new TextMatchBasedRewriteRule(connectContext, stmt, optToAstMap)
                         .transform(logicOperatorTree, context).get(0);
             }
 
-            // prepare for mv rewrite
-            prepareMvRewrite(connectContext, logicOperatorTree, columnRefFactory, requiredColumns);
-
-            logicOperatorTree = new TextMatchBasedRewriteRule(connectContext, stmt, optToAstMap)
-                    .transform(logicOperatorTree, context).get(0);
-
             OptExpression result = optimizerConfig.isRuleBased() ?
                     optimizeByRule(logicOperatorTree, requiredProperty, requiredColumns) :
                     optimizeByCost(connectContext, logicOperatorTree, requiredProperty, requiredColumns);