azagrebin commented on a change in pull request #13265:
URL: https://github.com/apache/flink/pull/13265#discussion_r478886087



##########
File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/memory/MemoryManager.java
##########
@@ -89,12 +89,14 @@
         *
         * @param memorySize The total size of the off-heap memory to be 
managed by this memory manager.
         * @param pageSize The size of the pages handed out by the memory 
manager.
+        * @param verifyEmptyWaitGcMaxSleeps defines how long to wait for GC of 
all allocated memory to check for memory leaks,
+        *                                   see also {@link 
UnsafeMemoryBudget}.

Review comment:
       You are right, I just did not want to repeat all details here, so I 
referred to `UnsafeMemoryBudget` for full description. At the end, the number 
of retries translates into the waiting time anyways.




----------------------------------------------------------------
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.

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


Reply via email to