Github user sbcd90 commented on the pull request:
https://github.com/apache/flink/pull/1566#issuecomment-179599473
Hello @tillrohrmann ..I have made all the changes you have mentioned.
- support for CompositeType.
- added test cases for both primitive type & composite type
- modified the changed test case in DataSinkTest.java
- changed the name of Comparator to ObjectArrayComparator.java
- javadocs are auto-generated by intellij.
Please please please please review my code now. Kindly let me know if there
are any further changes required. Kindly let me know if this can be merged now
also.
---
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.
---
