Skip to content

Commit

Permalink
[MINOR] improvement(server): LOG the exception call stack while unreg…
Browse files Browse the repository at this point in the history
…isterShuffle (#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.
  • Loading branch information
maobaolong authored Nov 11, 2024
1 parent 77c5c61 commit 7ca0719
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -172,6 +172,7 @@ public void unregisterShuffleByAppId(
}
} catch (Exception e) {
status = StatusCode.INTERNAL_ERROR;
LOG.error("App {} exception while unregisterShuffleByAppId", appId, e);
}

auditContext.withStatusCode(status);
Expand Down Expand Up @@ -215,6 +216,7 @@ public void unregisterShuffle(
}
} catch (Exception e) {
status = StatusCode.INTERNAL_ERROR;
LOG.error("App {} exception while unregisterShuffle", appId, e);
}

auditContext.withStatusCode(status);
Expand Down

0 comments on commit 7ca0719

Please sign in to comment.