XComp commented on a change in pull request #13316:
URL: https://github.com/apache/flink/pull/13316#discussion_r489477777



##########
File path: 
flink-runtime/src/test/java/org/apache/flink/runtime/io/network/buffer/NetworkBufferPoolTest.java
##########
@@ -109,6 +115,49 @@ public void testCreatePoolAfterDestroy() {
 
        }
 
+       @Test
+       public void testMemoryUsageProperties() {

Review comment:
       I split the test up into two test cases. The memory usage in the context 
of pool creation is already implicitly covered by 
`testMemoryUsageInTheContextOfMemorySegmentAllocation`.




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