I agree with your above points. 1. It is better to cover both of them, that means we cover most of the APIs from `RecordWriter` side. If necessary, I can submit a separate PR covering `BroadcastPartitioner` later.
2. I think your said is right. I just pointed out it because I did not see any tests for broadcast before. Your work supplements my previous PR, not duplicate with my work. :) [ Full content available at: https://github.com/apache/flink/pull/6697 ] This message was relayed via gitbox.apache.org for [email protected]
