Third time I'm looking at this PR and third time I had to think for a minute 
what does the this method. I'm always forgetting that `serializer` is a class 
field and that this method copies from it.

Maybe rename to `copyFromSerializerToTargetChannel`? Imo rename would allow us 
to drop most of the java doc and simplify it to just:
 ```
        /**
         * @param targetChannel
         * @return <tt>true</tt> if the intermediate serialization buffer 
should be pruned
         */
```

[ Full content available at: https://github.com/apache/flink/pull/6417 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to