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

roryqi 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 6555280b4 [#2082] fix: spark executor task error when reading shuffle 
data when using java open jdk11 (#2436)
6555280b4 is described below

commit 6555280b428a55a889f20dc76bdc555a85f1a9f1
Author: roryqi <ror...@apache.org>
AuthorDate: Thu Apr 10 00:09:27 2025 +0800

    [#2082] fix: spark executor task error when reading shuffle data when using 
java open jdk11 (#2436)
    
    ### What changes were proposed in this pull request?
    
    Remove unnecessary release for ByteBuf
    
    ### Why are the changes needed?
    
    Fix: #2082
    
    ### Does this PR introduce _any_ user-facing change?
    No.
    
    ### How was this patch tested?
    CI passed. User verified this fix in the production environment.
---
 .../java/org/apache/uniffle/client/impl/ShuffleReadClientImpl.java  | 6 ------
 1 file changed, 6 deletions(-)

diff --git 
a/client/src/main/java/org/apache/uniffle/client/impl/ShuffleReadClientImpl.java
 
b/client/src/main/java/org/apache/uniffle/client/impl/ShuffleReadClientImpl.java
index e1aa0f958..6a1701c95 100644
--- 
a/client/src/main/java/org/apache/uniffle/client/impl/ShuffleReadClientImpl.java
+++ 
b/client/src/main/java/org/apache/uniffle/client/impl/ShuffleReadClientImpl.java
@@ -310,9 +310,6 @@ public class ShuffleReadClientImpl implements 
ShuffleReadClient {
     if (sdr == null) {
       return 0;
     }
-    if (readBuffer != null) {
-      RssUtils.releaseByteBuffer(readBuffer);
-    }
     readBuffer = sdr.getDataBuffer();
     if (readBuffer == null || readBuffer.capacity() == 0) {
       return 0;
@@ -331,9 +328,6 @@ public class ShuffleReadClientImpl implements 
ShuffleReadClient {
     if (sdr != null) {
       sdr.release();
     }
-    if (readBuffer != null) {
-      RssUtils.releaseByteBuffer(readBuffer);
-    }
     if (clientReadHandler != null) {
       clientReadHandler.close();
     }

Reply via email to