zhiqiang-hhhh commented on code in PR #33690:
URL: https://github.com/apache/doris/pull/33690#discussion_r1690738219


##########
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:
   fixed



##########
fe/fe-common/src/main/java/org/apache/doris/common/Config.java:
##########
@@ -2694,6 +2694,10 @@ public class Config extends ConfigBase {
     @ConfField(mutable = true)
     public static boolean enable_cooldown_replica_affinity = true;
 
+    // The max number of profiles that can be stored to storage.
+    @ConfField
+    public static int max_spilled_profile_num = 500;

Review Comment:
   fixed



-- 
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