(uniffle) branch master updated: [Improvement] Clear the alarm log when starting the relevant service (#2453)

[yl09099]

This is an automated email from the ASF dual-hosted git repository.

yl09099 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new 4064da22d [Improvement] Clear the alarm log when starting the relevant 
service (#2453)
4064da22d is described below

commit 4064da22df62c4067fdbc2c5b5710179fcd660bf
Author: yl09099 <33595968+yl09...@users.noreply.github.com>
AuthorDate: Thu Apr 24 20:10:37 2025 +0800

    [Improvement] Clear the alarm log when starting the relevant service (#2453)
    
    * [Improvement] Clear the alarm service when the service starts.
    
    Signed-off-by: yl09099 <yl09...@apache.org>
---
 bin/start-coordinator.sh                                                | 1 +
 common/src/main/java/org/apache/uniffle/common/web/JettyServer.java     | 2 ++
 .../java/org/apache/uniffle/common/web/resource/MetricResource.java     | 1 -
 dashboard/src/main/java/org/apache/uniffle/dashboard/web/Dashboard.java | 2 ++
 4 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/bin/start-coordinator.sh b/bin/start-coordinator.sh
index a7211b317..66330fc11 100755
--- a/bin/start-coordinator.sh
+++ b/bin/start-coordinator.sh
@@ -93,6 +93,7 @@ GC_LOG_ARGS_NEW=" -XX:+IgnoreUnrecognizedVMOptions \
           -Xlog:gc+phases=debug \
           -Xlog:gc+ref=debug \
           -Xlog:gc+start=debug \
+          --add-opens java.base/java.lang=ALL-UNNAMED \
           -Xlog:gc*:file=${RSS_LOG_DIR}/gc-%t.log:tags,uptime,time,level"
 
 JVM_LOG_ARGS=""
diff --git 
a/common/src/main/java/org/apache/uniffle/common/web/JettyServer.java 
b/common/src/main/java/org/apache/uniffle/common/web/JettyServer.java
index 557ecf95a..ac357a6cb 100644
--- a/common/src/main/java/org/apache/uniffle/common/web/JettyServer.java
+++ b/common/src/main/java/org/apache/uniffle/common/web/JettyServer.java
@@ -155,6 +155,8 @@ public class JettyServer {
     resourcePackages.addAll(Arrays.asList(packages));
     servletHolder.setInitParameter(
         ServerProperties.PROVIDER_PACKAGES, String.join(",", 
resourcePackages));
+    // Turn off wadl.
+    servletHolder.setInitParameter(ServerProperties.WADL_FEATURE_DISABLE, 
"true");
   }
 
   public void registerInstance(Class<?> clazz, Object instance) {
diff --git 
a/common/src/main/java/org/apache/uniffle/common/web/resource/MetricResource.java
 
b/common/src/main/java/org/apache/uniffle/common/web/resource/MetricResource.java
index 4940bfef0..6c9e5873c 100644
--- 
a/common/src/main/java/org/apache/uniffle/common/web/resource/MetricResource.java
+++ 
b/common/src/main/java/org/apache/uniffle/common/web/resource/MetricResource.java
@@ -38,7 +38,6 @@ public class MetricResource extends BaseMetricResource {
   @Context protected ServletContext servletContext;
 
   @GET
-  @Path("/")
   public MetricsJsonObj metrics(@QueryParam("name[]") Set<String> names) {
     return metrics(null, names);
   }
diff --git 
a/dashboard/src/main/java/org/apache/uniffle/dashboard/web/Dashboard.java 
b/dashboard/src/main/java/org/apache/uniffle/dashboard/web/Dashboard.java
index bb8d1e25b..a4d510996 100644
--- a/dashboard/src/main/java/org/apache/uniffle/dashboard/web/Dashboard.java
+++ b/dashboard/src/main/java/org/apache/uniffle/dashboard/web/Dashboard.java
@@ -121,6 +121,8 @@ public class Dashboard {
     ServletHolder servletHolder = 
contextHandler.addServlet(ServletContainer.class, "/*");
     servletHolder.setInitParameter(
         ServerProperties.PROVIDER_PACKAGES, 
"org.apache.uniffle.dashboard.web.resource");
+    // Turn off wadl.
+    servletHolder.setInitParameter(ServerProperties.WADL_FEATURE_DISABLE, 
"true");
     contextHandler.setAttribute("coordinatorServerAddresses", 
coordinatorServerAddresses);
     contextHandler.setAttribute(Dashboard.class.getCanonicalName(), this);
     return contextHandler;