liuml07 commented on code in PR #24205:
URL: https://github.com/apache/flink/pull/24205#discussion_r2191622168


##########
flink-core/src/test/java/org/apache/flink/api/java/ClosureCleanerTest.java:
##########
@@ -133,6 +135,27 @@ void testComplexTopLevelClassClean() throws Exception {
         assertThat(result).isEqualTo(5);
     }
 
+    @Test
+    public void testCleanNonSerializableNestedMap() throws Exception {

Review Comment:
   There are multiple classes here in the test: `ComplexMap` has `map1` field 
which is a `MapFunction`. This MapFunction refers to a non-serializable Object. 
Meanwhile `ComplexMap` also has `map4` field  which is a `LocalMap`. This 
LocalMap also refers to same value of `map1`. So it has 4 layers of references.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to