EmmyMiao87 commented on a change in pull request #2821: The new materialized view selector URL: https://github.com/apache/incubator-doris/pull/2821#discussion_r375122934
########## File path: fe/src/main/java/org/apache/doris/qe/SessionVariable.java ########## @@ -90,6 +90,8 @@ public static final String LOAD_MEM_LIMIT = "load_mem_limit"; public static final String DEFAULT_ROWSET_TYPE = "default_rowset_type"; public static final String USE_V2_ROLLUP = "use_v2_rollup"; + public static final String TEST_MATERIALIZED_VIEW = "test_materialized_view"; Review comment: This properties has two functions. One is enable the `Create Materialized View Stmt`. The another effect is that if the index selected by the new selector is inconsistent with the old one, an error is reported during the test. So which name is better? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org