flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [flink] zhijiangW commented on issue #7713: [FLINK-10995][network] Copy intermediate serialization results only once for broadcast mode
Date Mon, 04 Mar 2019 04:01:12 GMT
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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message