This is an automated email from the ASF dual-hosted git repository.
nicholasjiang pushed a commit to branch branch-0.6
in repository https://gitbox.apache.org/repos/asf/celeborn.git
The following commit(s) were added to refs/heads/branch-0.6 by this push:
new 4d96eae1c [CELEBORN-2133] LifecycleManager should log stack trace of
Throwable for invoking appShuffleTrackerCallback
4d96eae1c is described below
commit 4d96eae1c70dca1c12b8a2feb859500ce11589ec
Author: SteNicholas <[email protected]>
AuthorDate: Wed Aug 27 10:31:56 2025 +0800
[CELEBORN-2133] LifecycleManager should log stack trace of Throwable for
invoking appShuffleTrackerCallback
### What changes were proposed in this pull request?
`LifecycleManager` should log stack trace of `Throwable` for invoking
`appShuffleTrackerCallback`.
### Why are the changes needed?
`LifecycleManager` does not log stack trace of `Throwable` for invoking
`appShuffleTrackerCallback` at present, which log is as follows:
```
ERROR LifecycleManager: java.lang.RuntimeException
```
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
CI.
Closes #3453 from SteNicholas/CELEBORN-2133.
Authored-by: SteNicholas <[email protected]>
Signed-off-by: SteNicholas <[email protected]>
(cherry picked from commit 20e36ca72d668db67dcd8ebbc785632b3f0755f4)
Signed-off-by: SteNicholas <[email protected]>
---
client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
b/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
index be93a00c7..b049e0932 100644
--- a/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
+++ b/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
@@ -1069,7 +1069,7 @@ class LifecycleManager(val appUniqueId: String, val conf:
CelebornConf) extends
true
} catch {
case t: Throwable =>
- logError(t.toString)
+ logError("Failed to invoke appShuffleTrackerCallback.", t)
false
}
case None =>
