igalshilman commented on a change in pull request #7590: [FLINK-11329][core] Migrating CompositeSerializers URL: https://github.com/apache/flink/pull/7590#discussion_r252139840
########## File path: flink-end-to-end-tests/flink-stream-state-ttl-test/src/main/java/org/apache/flink/streaming/tests/verify/ValueWithTs.java ########## @@ -56,8 +57,8 @@ public String toString() { /** Serializer for Serializer. */ public static class Serializer extends CompositeSerializer<ValueWithTs<?>> { - public Serializer(TypeSerializer<?> userValueSerializer) { - super(true, userValueSerializer, LongSerializer.INSTANCE); + public Serializer(TypeSerializer<?> valueSerializer, TypeSerializer<Long> timestampSerializer) { Review comment: As far as I can tell this is not a user facing serializer, rather used internally by the `TtlSerializer`, and I think it is important to make it explicit that this is a composite serializer and these are the nested serializers that define it. As a way to reduce verbosity, we can add a static factory method with an explicit name. Feel free to decide for yourself :-) ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services