Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4688
Thanks for your review @tzulitai. Travis passed. Will merge the PR.
---
Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/4688
@tillrohrmann Yes that makes sense, lets do that separately in the future
then.
+1.
---
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4688
Thanks for the review @tzulitai. I've addressed your comments. Concerning
the custom serialization logic, I think it does not hurt to have it, because it
should be more efficient than the
