fix comment

src/common/src/config.rs

@@ -924,10 +924,6 @@ pub mod default {
         pub fn telemetry_enabled() -> bool {
             true
         }
-
-        pub fn manually_dump_heap_profile_dir() -> String {
-            "".to_string()
-        }
     }
 
     pub mod storage {

src/common/src/heap_profiling/mod.rs

@@ -12,8 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-pub const MANUALLY_DUMP_MID_NAME: &str = "manually-dump.compute.dump";
-pub const AUTO_DUMP_MID_NAME: &str = "auto-dump.compute.dump";
+pub const MANUALLY_DUMP_MID_NAME: &str = "manually-dump.compute.heap";
+pub const AUTO_DUMP_MID_NAME: &str = "auto-dump.compute.heap";
 pub const COLLAPSED_SUFFIX: &str = "collapsed";
 
 pub mod jeprof;

src/compute/src/lib.rs

@@ -125,7 +125,7 @@ pub struct ComputeNodeOpts {
     /// Enable heap profile dump when memory usage is high.
     #[clap(long, env = "RW_HEAP_PROFILING_DIR")]
     #[override_opts(path = server.heap_profiling.dir)]
-    pub heap_profiling: Option<String>,
+    pub heap_profiling_dir: Option<String>,
 
     #[clap(long, env = "RW_OBJECT_STORE_STREAMING_READ_TIMEOUT_MS", value_enum)]
     #[override_opts(path = storage.object_store_streaming_read_timeout_ms)]

src/compute/src/rpc/service/monitor_service.rs

@@ -140,7 +140,7 @@ impl MonitorService for MonitorServiceImpl {
         }
 
         let time_prefix = chrono::Local::now().format("%Y-%m-%d-%H-%M-%S").to_string();
-        let file_name = format!("{}.{}\0", time_prefix, AUTO_DUMP_MID_NAME);
+        let file_name = format!("{}.{}\0", time_prefix, MANUALLY_DUMP_MID_NAME);
         let arg_dir = request.into_inner().get_dir().clone();
         let dir = PathBuf::from(if arg_dir.is_empty() {
             &self.server_config.heap_profiling.dir