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 80d601a10 [CELEBORN-2171] Fix array index error in 
submitRetryPushMergedData
80d601a10 is described below

commit 80d601a1058187fbb7e620a8c5b00e32fd78f1b2
Author: xxx <[email protected]>
AuthorDate: Mon Oct 13 11:44:58 2025 +0800

    [CELEBORN-2171] Fix array index error in submitRetryPushMergedData
    
    ### What changes were proposed in this pull request?
    
    Fix array index error in `submitRetryPushMergedData`.
    
    ### Why are the changes needed?
    
    Fix array index error in `submitRetryPushMergedData`.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    CI.
    
    Closes #3501 from xy2953396112/CELEBORN-2171.
    
    Authored-by: xxx <[email protected]>
    Signed-off-by: SteNicholas <[email protected]>
    (cherry picked from commit c1f73a3d77fe34322773ca9dc32131baa7f5e8e3)
    Signed-off-by: SteNicholas <[email protected]>
---
 client/src/main/java/org/apache/celeborn/client/ShuffleClientImpl.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/client/src/main/java/org/apache/celeborn/client/ShuffleClientImpl.java 
b/client/src/main/java/org/apache/celeborn/client/ShuffleClientImpl.java
index 4dd060074..cfe40a296 100644
--- a/client/src/main/java/org/apache/celeborn/client/ShuffleClientImpl.java
+++ b/client/src/main/java/org/apache/celeborn/client/ShuffleClientImpl.java
@@ -476,7 +476,7 @@ public class ShuffleClientImpl extends ShuffleClient {
     }
 
     for (int i = index; i < reviveRequests.length; i++) {
-      ReviveRequest request = reviveRequests[index];
+      ReviveRequest request = reviveRequests[i];
       DataBatches.DataBatch batch = batches.get(i);
       if (remainReviveTimes > 0) {
         reviveFailedBatchesMap.add(batch);

Reply via email to