(incubator-uniffle) branch master updated: [MINOR] improvement(server): LOG the exception call stack while unregisterShuffle (#2228)

[zuston]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7ca07196e [MINOR] improvement(server): LOG the exception call stack 
while unregisterShuffle (#2228)
7ca07196e is described below

commit 7ca07196ef7fb960404ffca5b62f1e53227f6c37
Author: maobaolong <baoloong...@tencent.com>
AuthorDate: Mon Nov 11 16:48:36 2024 +0800

    [MINOR] improvement(server): LOG the exception call stack while 
unregisterShuffle (#2228)
    
    ### What changes were proposed in this pull request?
    
    LOG the exception call stack while unregisterShuffle
    
    ### Why are the changes needed?
    
    Easy to find the error call stack.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    No need.
---
 .../main/java/org/apache/uniffle/server/ShuffleServerGrpcService.java   | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/server/src/main/java/org/apache/uniffle/server/ShuffleServerGrpcService.java 
b/server/src/main/java/org/apache/uniffle/server/ShuffleServerGrpcService.java
index 8c877c95d..0c38960d0 100644
--- 
a/server/src/main/java/org/apache/uniffle/server/ShuffleServerGrpcService.java
+++ 
b/server/src/main/java/org/apache/uniffle/server/ShuffleServerGrpcService.java
@@ -172,6 +172,7 @@ public class ShuffleServerGrpcService extends 
ShuffleServerImplBase {
         }
       } catch (Exception e) {
         status = StatusCode.INTERNAL_ERROR;
+        LOG.error("App {} exception while unregisterShuffleByAppId", appId, e);
       }
 
       auditContext.withStatusCode(status);
@@ -215,6 +216,7 @@ public class ShuffleServerGrpcService extends 
ShuffleServerImplBase {
         }
       } catch (Exception e) {
         status = StatusCode.INTERNAL_ERROR;
+        LOG.error("App {} exception while unregisterShuffle", appId, e);
       }
 
       auditContext.withStatusCode(status);