seawinde commented on code in PR #34418:
URL: https://github.com/apache/doris/pull/34418#discussion_r1597397908


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CreateMTMVInfo.java:
##########
@@ -88,7 +88,7 @@
  */
 public class CreateMTMVInfo {
     public static final Logger LOG = 
LogManager.getLogger(CreateMTMVInfo.class);
-    public static final String MTMV_PLANER_DISABLE_RULES = 
"OLAP_SCAN_PARTITION_PRUNE";
+    public static final String MTMV_PLANER_DISABLE_RULES = 
"OLAP_SCAN_PARTITION_PRUNE,PRUNE_EMPTY_PARTITION";

Review Comment:
   the mv plan used by query rewrite is generated by 
`org.apache.doris.mtmv.MTMVCache#from` which runs the same rules as query.
   `MTMV_PLANER_DISABLE_RULES` is only used by mv meta check when create mtmv. 
it seems has no problems



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to