Jibing-Li commented on code in PR #31276: URL: https://github.com/apache/doris/pull/31276#discussion_r1498935181
########## fe/fe-common/src/main/java/org/apache/doris/common/Config.java: ########## @@ -1902,6 +1902,10 @@ public class Config extends ConfigBase { "Max cache number of remote file system."}) public static long max_remote_file_system_cache_num = 100; + @ConfField(mutable = false, masterOnly = false, description = {"外表行数缓存最大数量", + "Max cache number of external table row count"}) + public static long max_external_table_row_count_cache_num = 100000; Review Comment: There's multiple config items for external table cache, such as max_remote_file_system_cache_num, max_external_cache_loader_thread_pool_size and so on. How about keep this pr as it is , and move all the related configs to session variable in a new pr if we need to do so in the future. -- 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