azagrebin commented on a change in pull request #12981: URL: https://github.com/apache/flink/pull/12981#discussion_r460103524
########## File path: flink-runtime/src/main/java/org/apache/flink/runtime/memory/UnsafeMemoryBudget.java ########## @@ -147,8 +147,10 @@ void reserveMemory(long size, int maxSleeps) throws MemoryReservationException { } // no luck - throw new MemoryReservationException( - String.format("Could not allocate %d bytes, only %d bytes are remaining", size, availableOrReserved)); + throw new MemoryReservationException(String.format( + "Could not allocate %d bytes, only %d bytes are remaining, try to upgrade to Java 8u72 or higher", Review comment: The warning can be overlooked. We agreed with @tillrohrmann that we want to duplicate it here if the warning has been overlooked. Maybe, we could add a bit details why to upgrade or ask `JavaGcCleanerWrapper` whether running cleaners is supported by the JVM in use to make the hint conditional. ---------------------------------------------------------------- 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