[GitHub] flink issue #5735: [FLINK-9036] [core] Add default values to State Descripto...

2018-05-16 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/5735 I think I'd favour the second approach because the default value is closer to where the return value is used. If it's define "far away" when defining the state descriptor it's easier for code to get

[GitHub] flink issue #5735: [FLINK-9036] [core] Add default values to State Descripto...

2018-05-11 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5735 @aljoscha Would be interested in your opinion here. This is basically one of two ways to improve the handling of default values: 1. Add a default value supplier on the state descript