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

    https://github.com/apache/flink/pull/5732#discussion_r176222855
  
    --- Diff: 
flink-core/src/test/java/org/apache/flink/api/common/state/StateDescriptorTest.java
 ---
    @@ -130,6 +135,31 @@ public void 
testInitializeSerializerAfterSerializationWithCustomConfig() throws
                                .getRegistration(File.class).getId() > 0);
        }
     
    +   // 
------------------------------------------------------------------------
    +   //  Tests for serializer initialization
    +   // 
------------------------------------------------------------------------
    +
    +   /**
    +    * FLINK-6775, tests that the returned serializer is duplicated.
    +    * This allows to share the state descriptor across threads.
    +    */
    +   @Test
    +   public void testSerializerDuplication() throws Exception {
    +           // we need a serializer that actually duplicates for testing (a 
stateful one)
    --- End diff --
    
    Same as above, we should assert that assumption


---

Reply via email to