[GitHub] flink issue #5732: [FLINK-9034] [FLINK-9035] [core] Fix state descriptors

2018-04-02 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5732 Manually merged in f3a519712fb31f7b71181e876c3c3d5fff08eb71 ---

[GitHub] flink issue #5732: [FLINK-9034] [FLINK-9035] [core] Fix state descriptors

2018-03-22 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/5732 @StephanEwen Yes, I think this should go into 1.5.0 because it fixes potential (and real) problems. And yes, I wasn't suggesting to remove `initializeSerializerUnlessSet(ExecutionConfig)` now, but

[GitHub] flink issue #5732: [FLINK-9034] [FLINK-9035] [core] Fix state descriptors

2018-03-22 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5732 Concerning guarding the test assumptions: I think it is fine, because if the KryoSerializer is not duplicating properly any more (assumption violated), the test will also fail. (The problematic