[apache#1628] test(client): Fix double release ShuffleIndexResult cau…

…se exception to pass flaky test (apache#2179)

### What changes were proposed in this pull request?

(Please outline the changes and how this PR fixes the issue.)

### Why are the changes needed?

- Pass the flaky test
- Avoid memory leak after double release the `ShuffleIndexResult`

Fix: apache#1628

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Flaky test can be fixed.

(cherry picked from commit 4c14b97)

common/src/main/java/org/apache/uniffle/common/ShuffleIndexResult.java

@@ -20,12 +20,17 @@
 import java.nio.ByteBuffer;
 
 import io.netty.buffer.Unpooled;
+import io.netty.util.IllegalReferenceCountException;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import org.apache.uniffle.common.netty.buffer.ManagedBuffer;
 import org.apache.uniffle.common.netty.buffer.NettyManagedBuffer;
 import org.apache.uniffle.common.util.ByteBufUtils;
 
 public class ShuffleIndexResult {
+  private static final Logger LOG = LoggerFactory.getLogger(ShuffleIndexResult.class);
+
   private final ManagedBuffer buffer;
   private long dataFileLen;
   private String dataFileName;
@@ -74,7 +79,16 @@ public boolean isEmpty() {
 
   public void release() {
     if (this.buffer != null) {
-      this.buffer.release();
+      try {
+        this.buffer.release();
+      } catch (IllegalReferenceCountException e) {
+        LOG.warn(
+            "Failed to release shuffle index result with length {} of {}. "
+                + "Maybe it has been released by others.",
+            dataFileLen,
+            dataFileName,
+            e);
+      }
     }
   }