(uniffle) branch master updated: [MINOR] improvement(remote merge): print log when merge failed (#2479)

[xianjingfeng]

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

xianjingfeng 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 855cec120 [MINOR] improvement(remote merge): print log when merge 
failed (#2479)
855cec120 is described below

commit 855cec120382f68864b353f7fba501cf63947672
Author: xianjingfeng <xianjingfeng...@gmail.com>
AuthorDate: Fri May 9 17:18:06 2025 +0800

    [MINOR] improvement(remote merge): print log when merge failed (#2479)
    
    ### What changes were proposed in this pull request?
    Print log when merge failed
    
    ### Why are the changes needed?
    Currently we don't know the reason why merge failed from the logs.
    
    ### Does this PR introduce any user-facing change?
    No.
    
    ### How was this patch tested?
    Manual Testing
---
 .../main/java/org/apache/uniffle/server/merge/ShuffleMergeManager.java  | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/server/src/main/java/org/apache/uniffle/server/merge/ShuffleMergeManager.java 
b/server/src/main/java/org/apache/uniffle/server/merge/ShuffleMergeManager.java
index 4de51df3b..57e3ee680 100644
--- 
a/server/src/main/java/org/apache/uniffle/server/merge/ShuffleMergeManager.java
+++ 
b/server/src/main/java/org/apache/uniffle/server/merge/ShuffleMergeManager.java
@@ -282,6 +282,8 @@ public class ShuffleMergeManager {
       // 5 merge segments to output
       partition.merge(segments, output, reader);
       success = true;
+    } catch (Throwable e) {
+      LOG.error("Merge failed, caused by ", e);
     } finally {
       if (!success && partition != null) {
         partition.setState(INTERNAL_ERROR);