Github user sihuazhou commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5732#discussion_r175983599
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/api/common/state/StateDescriptor.java 
---
    @@ -77,18 +80,22 @@
     
        /** The serializer for the type. May be eagerly initialized in the 
constructor,
         * or lazily once the type is serialized or an ExecutionConfig is 
provided. */
    +   @Nullable
        protected TypeSerializer<T> serializer;
     
    +   /** The type information describing the value type. Only used to lazily 
create the serializer
    +    * and dropped during serialization */
    +   @Nullable
    --- End diff --
    
    nit:Type information will not dropped during serialization now, it dropped 
in `initializeSerializerUnlessSet()`.


---

Reply via email to