[apache#933] fix incorrect metric grpc_server_connection_number (apac…

…he#934) ### What changes were proposed in this pull request? Fix incorrect metric grpc_server_connection_number ### Why are the changes needed? Fix: apache#933 ### Does this PR introduce any user-facing change? No. ### How was this patch tested? Existing UT
xianjingfeng · Jun 8, 2023 · b6a725b · b6a725b
1 parent cb22510
commit b6a725b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/common/src/main/java/org/apache/uniffle/common/rpc/MonitoringServerTransportFilter.java b/common/src/main/java/org/apache/uniffle/common/rpc/MonitoringServerTransportFilter.java
@@ -37,7 +37,9 @@ public Attributes transportReady(Attributes transportAttrs) {
   }
 
   public void transportTerminated(Attributes transportAttrs) {
-    grpcMetrics.decGauge(GRPC_SERVER_CONNECTION_NUMBER_KEY);
+    if (transportAttrs != null) {
+      grpcMetrics.decGauge(GRPC_SERVER_CONNECTION_NUMBER_KEY);
+    }
     super.transportTerminated(transportAttrs);
   }
 }