curcur commented on pull request #12912: URL: https://github.com/apache/flink/pull/12912#issuecomment-660627891
1. Yes, I like Option 1. It looks very strange that two threads are trying to release the network resources at the same time. 2. Wondering whether releasing the input channel in the same order in both places would resolve the problem as well, but not sure how difficult in this case. 3. The code change seems to based on an old interface, and the new one is like this: ---------------------------------------------------------------- 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: us...@infra.apache.org