[GitHub] [doris] Jibing-Li commented on a diff in pull request #20906: [Improvement](multi catalog, statistics)Support two level external statistics cache loader.

["Jibing-Li (via GitHub)" <gi...@apache.org>]

Jibing-Li commented on code in PR #20906:
URL: https://github.com/apache/doris/pull/20906#discussion_r1233448879


##########
fe/fe-common/src/main/java/org/apache/doris/common/Config.java:
##########
@@ -1753,7 +1753,7 @@ public class Config extends ConfigBase {
      * Otherwise, use external catalog metadata.
      */
     @ConfField(mutable = true)
-    public static boolean collect_external_table_stats_by_sql = false;
+    public static boolean collect_external_table_stats_by_sql = true;

Review Comment:
   Will remove this in next PR. Next PR will move the code of fetching statistics from data source to TableIf, and will remove this config.



-- 
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-unsubscribe@doris.apache.org

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


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