Address comment

apache · Nov 16, 2024 · 0b12740 · 0b12740
1 parent dcac751
commit 0b12740
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/server/src/main/java/org/apache/uniffle/server/ShuffleServerConf.java b/server/src/main/java/org/apache/uniffle/server/ShuffleServerConf.java
@@ -719,8 +719,8 @@ public class ShuffleServerConf extends RssBaseConf {
           .booleanType()
           .defaultValue(false)
           .withDescription("Whether to enable app detail log");
-  public static final ConfigOption<Boolean> SERVER_TRIGGER_REPORT_WHILE_UNREGISTER_ENABLED =
-      ConfigOptions.key("rss.server.trigger.report.while.unregister.enabled")
+  public static final ConfigOption<Boolean> SERVER_HEARTBEAT_REPORT_ON_UNREGISTER_ENABLED =
+      ConfigOptions.key("rss.server.heartbeatReportOnUnregisterEnabled")
           .booleanType()
           .defaultValue(false)
           .withDescription("Whether to trigger report while unregister");

diff --git a/server/src/main/java/org/apache/uniffle/server/ShuffleServerGrpcService.java b/server/src/main/java/org/apache/uniffle/server/ShuffleServerGrpcService.java
@@ -164,7 +164,7 @@ public void unregisterShuffleByAppId(
       try {
         if (shuffleServer
             .getShuffleServerConf()
-            .getBoolean(ShuffleServerConf.SERVER_RPC_AUDIT_LOG_ENABLED)) {
+            .getBoolean(ShuffleServerConf.SERVER_HEARTBEAT_REPORT_ON_UNREGISTER_ENABLED)) {
           shuffleServer.sendHeartbeat();
         }
         shuffleServer.getShuffleTaskManager().removeShuffleDataAsync(appId);