[GitHub] flink pull request: [FLINK-2936] Fix ClassCastException for Event-...

2015-12-11 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/1448#issuecomment-163918357 @StephanEwen do we want to fix it like this now, or do we want to integrate it into the StreamRecordSerializer? --- If your project is set up for it, you can reply to

[GitHub] flink pull request: [FLINK-2936] Fix ClassCastException for Event-...

2015-12-11 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1448#issuecomment-163940693 Probably okay to do it like this. How hard is it to pass the "timestamps" flag into the constructor of the SourceOperator, rather than through the runtime

[GitHub] flink pull request: [FLINK-2936] Fix ClassCastException for Event-...

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

[GitHub] flink pull request: [FLINK-2936] Fix ClassCastException for Event-...

2015-12-09 Thread aljoscha
GitHub user aljoscha opened a pull request: https://github.com/apache/flink/pull/1448 [FLINK-2936] Fix ClassCastException for Event-Time source Before, would throw a ClassCastException when emitting watermarks with timestamp/watermark multiplexing disabled. You can merge this