You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2019/01/22 10:54:07 UTC

[GitHub] tzulitai opened a new pull request #7553: [FLINK-11328] [core] Migrate all parameterless serializers to use new serialization compatibility API

tzulitai opened a new pull request #7553: [FLINK-11328] [core] Migrate all parameterless serializers to use new serialization compatibility API
URL: https://github.com/apache/flink/pull/7553
 
 
   ## What is the purpose of the change
   
   This PR migrates all parameterless serializers (in Flink, they happen to all be subclasses of `TypeSerializerSingleton`) to use the new serialization compatibility APIs, so that the serializers themselves are no longer Java-serialized into savepoints.
   
   ## Brief change log
   
   - edf6d59: This commit essentially does 2 things: 1) improve usability of the new `SimpleTypeSerializerSnapshot` base class, so that it may be used by serializers that have varying ways for instantiation, and 2) let all subclasses of `TypeSerializerSingleton` return their own implementation of a `SimpleTypeSerializerSnapshot`, while removing the base implementation of `snapshotConfiguration` in `TypeSerializerSingleton`.
   - 898f6c5: Some CEP serializers for NFA data structures use nested serializers, so they should actually use the `CompositeTypeSerializerSnapshot` snapshot class. This commit addresses that.
   - 74adb84 to b4910d4: Some cleanup in tests.
   - 79c85ca to 18187c0: Commits that add migration tests for the touched serializers.
   
   ## Verifying this change
   
   All new subclasses of `TypeSerializerSnapshotMigrationTestBase` should pass.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services