dawidwys commented on a change in pull request #15055:
URL: https://github.com/apache/flink/pull/15055#discussion_r657841664
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/LocalInputChannel.java
##########
@@ -271,15 +271,22 @@ private ResultSubpartitionView
checkAndWaitForSubpartitionView() {
@Override
public void resumeConsumption() {
- checkState(!isReleased, "Channel released.");
Review comment:
Wouldn't the other option be as simple as adding a method:
```
interface BarrierHandlerState {
BarrierHandlerState endOfPartitionReceived(
Controller controller,
InputChannelInfo channelInfo);
}
```
and removing such channel from the blocked ones?
I'd feel safer with that check in place.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
