Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2623
@chermenin Do you plan to continue with this pull request? Would be great
if you
I would suggest to adjust it in the following way:
- Update the Chill dependency (validate if they change any existing
serializer)
- Exclude Kryo version 2 and version 3 from the Chill dependency. That
way, Flink's Kryo version should be used by Chill as well.
Also, testing the serializers via a "GroupReduce" integration test seems
heavy and unspecific. Can you change this to a serializer Unit test? Have a
look at the Kryo Serializer unit tests for an example.
Do you think that would work?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
