yiguolei commented on code in PR #33690:
URL: https://github.com/apache/doris/pull/33690#discussion_r1689175687
##########
fe/fe-core/src/main/java/org/apache/doris/common/util/ProfileManager.java:
##########
@@ -72,9 +71,10 @@
* the purpose is let coordinator can destruct earlier(the fragment profile is
in Coordinator)
*
*/
-public class ProfileManager {
+public class ProfileManager extends MasterDaemon {
private static final Logger LOG =
LogManager.getLogger(ProfileManager.class);
private static volatile ProfileManager INSTANCE = null;
+ private static final String PROFILE_STORAGE_PATH = Config.audit_log_dir +
File.separator + "profile";
Review Comment:
这里默认值可以是这个，但是应该有单独的配置项，来单独设置profile 的存储路径，如果设置了，那么就不应该用这个默认的路径了
--
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
