[GitHub] flink pull request: Added ResultTypeQueryable interface to TypeSer...

2015-01-30 Thread aalexandrov
Github user aalexandrov commented on the pull request: https://github.com/apache/flink/pull/349#issuecomment-72284550 @hsaputra: I just did: [FLINK-1464](https://issues.apache.org/jira/browse/FLINK-1464). --- If your project is set up for it, you can reply to this email and have

[GitHub] flink pull request: Added ResultTypeQueryable interface to TypeSer...

2015-01-30 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/flink/pull/349#issuecomment-72262547 @aalexandrov, could you file JIRA to associate this PR with? I need it for tracking purposes and releases. Thanks! --- If your project is set up for it, you can

[GitHub] flink pull request: Added ResultTypeQueryable interface to TypeSer...

2015-01-29 Thread twalthr
Github user twalthr commented on the pull request: https://github.com/apache/flink/pull/349#issuecomment-72042165 The change makes sense. The TypeExtractor can never get the types from `new TypeSerializerInputFormatXYZ()` automatically, so passing them in the constructor is correct.