You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celeborn.apache.org by et...@apache.org on 2023/03/06 13:49:22 UTC

[incubator-celeborn] branch main updated: [CELEBORN-379][FLINK] Fix checkState in TransportFrameDecoderWithBufferSupplier#decodeBodyCopyOut (#1311)

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

ethanfeng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git


The following commit(s) were added to refs/heads/main by this push:
     new fd1ac2f71 [CELEBORN-379][FLINK] Fix checkState in TransportFrameDecoderWithBufferSupplier#decodeBodyCopyOut (#1311)
fd1ac2f71 is described below

commit fd1ac2f71179a1d17b814e5df2bd468a18176c6b
Author: Keyong Zhou <zh...@apache.org>
AuthorDate: Mon Mar 6 21:49:18 2023 +0800

    [CELEBORN-379][FLINK] Fix checkState in TransportFrameDecoderWithBufferSupplier#decodeBodyCopyOut (#1311)
---
 .../plugin/flink/network/TransportFrameDecoderWithBufferSupplier.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/network/TransportFrameDecoderWithBufferSupplier.java b/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/network/TransportFrameDecoderWithBufferSupplier.java
index ce85e7167..29657d419 100644
--- a/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/network/TransportFrameDecoderWithBufferSupplier.java
+++ b/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/network/TransportFrameDecoderWithBufferSupplier.java
@@ -124,7 +124,7 @@ public class TransportFrameDecoderWithBufferSupplier extends ChannelInboundHandl
     ReadData readData = (ReadData) curMsg;
     if (externalBuf == null) {
       Supplier<ByteBuf> supplier = bufferSuppliers.get(readData.getStreamId());
-      checkState(supplier == null, "Stream " + readData.getStreamId() + " buffer supplier is null");
+      checkState(supplier != null, "Stream " + readData.getStreamId() + " buffer supplier is null");
       externalBuf = bufferSuppliers.get(readData.getStreamId()).get();
     }
     copyByteBuf(buf, externalBuf, bodySize);