zhijiangW commented on issue #7713: [FLINK-10995][network] Copy intermediate 
serialization results only once for broadcast mode
URL: https://github.com/apache/flink/pull/7713#issuecomment-469110871
 
 
   @pnowojski , thanks for the kindly suggestion!
   
   The current broadcast usages for `RecordWriter` is only for emitting 
`Watermark` and `StreamStatus`.  Your worries might be right and I also ever 
considered that it might bring bad effects for special scenarios if switching 
frequently.
   
   Maybe the conservative way is only making the special improvement path for 
`BroadcastRecordWriter` which switches the mode in emitting `LatencyMarker` 
currently. If you approve this way, I would adjust the codes then. :)

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


With regards,
Apache Git Services

Reply via email to