Github user aljoscha commented on a diff in the pull request:
https://github.com/apache/flink/pull/5732#discussion_r176221943
--- Diff:
flink-core/src/main/java/org/apache/flink/api/common/state/StateDescriptor.java
---
@@ -249,12 +257,13 @@ public boolean isSerializerInitialized() {
*/
public void initializeSerializerUnlessSet(ExecutionConfig
executionConfig) {
--- End diff --
This is slightly orthogonal to this change, but: could we get rid of this
method and instead change `getSerializer()` to
`getSerializer(ExecutionConfig)`. That way, we don't have to be concerned about
forgetting to call `initializeSerializerUnlessSet()`.
---
