[GitHub] flink pull request: [FLINK-2906] Remove Record API

2015-11-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1403 --- 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 enab

[GitHub] flink pull request: [FLINK-2906] Remove Record API

2015-11-25 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/flink/pull/1403#issuecomment-159763961 Awesome! --- 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 enab

[GitHub] flink pull request: [FLINK-2906] Remove Record API

2015-11-25 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/1403#issuecomment-159747510 Will merge this PR. --- 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 fe

[GitHub] flink pull request: [FLINK-2906] Remove Record API

2015-11-25 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1403#issuecomment-159588695 Nice! Waiting for this for over a year now ;-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] flink pull request: [FLINK-2906] Remove Record API

2015-11-24 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/1403#issuecomment-159381208 Look good to me. I was about to prepare the same PR and did not find more files. I'm very happy that we finally get rid of this heritage :-) --- If your projec

[GitHub] flink pull request: [FLINK-2906] Remove Record API

2015-11-24 Thread zentol
GitHub user zentol opened a pull request: https://github.com/apache/flink/pull/1403 [FLINK-2906] Remove Record API This PR removes the old Record API and all Collector-Map related classes since they are unused now. I'm not sure i found all files, i went through the deprecat