[jira] [Created] (FLINK-13876) Remove ExecutionConfig field from PojoSerializer

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (FLINK-13876) Remove ExecutionConfig field from PojoSerializer

Shang Yuanchun (Jira)
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)