AHeise commented on a change in pull request #15771:
URL: https://github.com/apache/flink/pull/15771#discussion_r636012283



##########
File path: 
flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/io/RecordWriterOutput.java
##########
@@ -101,32 +97,51 @@ public void collect(StreamRecord<OUT> record) {
     }
 
     private <X> void pushToRecordWriter(StreamRecord<X> record) {
-        serializationDelegate.setInstance(record);
+        if (announcedStatus.isIdle()) {

Review comment:
       The comment is good in respect to explaining the invariant (you could 
also give a link to the docs). However, it's not helping the reader to 
understand why this case is happening.




-- 
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]


Reply via email to