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