Dawid Wysakowicz created FLINK-13876:
----------------------------------------

             Summary: Remove ExecutionConfig field from PojoSerializer
                 Key: FLINK-13876
                 URL: https://issues.apache.org/jira/browse/FLINK-13876
             Project: Flink
          Issue Type: Improvement
          Components: API / Type Serialization System
    Affects Versions: 1.9.0, 1.8.1, 1.7.2
            Reporter: Dawid Wysakowicz
             Fix For: 1.10.0


The PojoSerializers stores an instance of ExecutionConfig as internal field, 
even though that the only information it may ever need are the registered kryo 
serializers.

This has a few drawbacks:
* It blocks the evolution of {{ExecutionConfig}} as serializers where stored in 
a state. Therefore any change to ExecutionConfig must be backwards compatible 
in respect to java serialization
* It probably already introduced a bug, as upon restore the Snapshot actually 
recreates the serializer with an empty ExecutionConfig (see 
org.apache.flink.api.java.typeutils.runtime.PojoSerializerSnapshot#restoreSerializer)

I suggest to remove the field completely and adjust corresponding usages.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Reply via email to