You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iotdb.apache.org by ja...@apache.org on 2023/04/20 05:59:33 UTC
[iotdb] branch rel/1.1 updated: [To rel/1.1] Fix potential Npe of ShuffleSinkHandle
This is an automated email from the ASF dual-hosted git repository.
jackietien pushed a commit to branch rel/1.1
in repository https://gitbox.apache.org/repos/asf/iotdb.git
The following commit(s) were added to refs/heads/rel/1.1 by this push:
new b50469b39a [To rel/1.1] Fix potential Npe of ShuffleSinkHandle
b50469b39a is described below
commit b50469b39aab57791f9afd116f86db8920b602d0
Author: Jackie Tien <ja...@gmail.com>
AuthorDate: Thu Apr 20 13:59:26 2023 +0800
[To rel/1.1] Fix potential Npe of ShuffleSinkHandle
---
.../apache/iotdb/db/mpp/execution/exchange/sink/SinkChannel.java | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/server/src/main/java/org/apache/iotdb/db/mpp/execution/exchange/sink/SinkChannel.java b/server/src/main/java/org/apache/iotdb/db/mpp/execution/exchange/sink/SinkChannel.java
index af52f610fa..da575c2c01 100644
--- a/server/src/main/java/org/apache/iotdb/db/mpp/execution/exchange/sink/SinkChannel.java
+++ b/server/src/main/java/org/apache/iotdb/db/mpp/execution/exchange/sink/SinkChannel.java
@@ -50,6 +50,7 @@ import java.util.List;
import java.util.Map.Entry;
import java.util.concurrent.ExecutorService;
+import static com.google.common.util.concurrent.Futures.immediateVoidFuture;
import static com.google.common.util.concurrent.Futures.nonCancellationPropagating;
import static org.apache.iotdb.db.mpp.common.FragmentInstanceId.createFullId;
import static org.apache.iotdb.db.mpp.metric.DataExchangeCostMetricSet.SEND_NEW_DATA_BLOCK_EVENT_TASK_CALLER;
@@ -145,6 +146,11 @@ public class SinkChannel implements ISinkChannel {
@Override
public synchronized ListenableFuture<?> isFull() {
checkState();
+ // blocked could be null if this channel is closed before it is opened by ShuffleSinkHandle
+ // return immediateVoidFuture() to avoid NPE
+ if (closed) {
+ return immediateVoidFuture();
+ }
return nonCancellationPropagating(blocked);
}