Github user ggevay commented on the issue:

    https://github.com/apache/flink/pull/3511
  
    I pushed some more commits to 
[https://github.com/ggevay/flink/tree/sorter-codegen](https://github.com/ggevay/flink/tree/sorter-codegen).
 @heytitle , could you please push them here? (Or you could add me as a 
collaborator to your flink repo, and then I can directly push to your branch of 
the PR.)
    
    @heytitle , @greghogan , could you do a review? I think this PR is close to 
a mergable state. (I would say we should merge this without 
`FixedLengthRecordSorter`, and then do that later after 
https://github.com/apache/flink/pull/2617 is merged.)


---
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.
---

Reply via email to