[GitHub] [geode] dschneider-pivotal commented on a change in pull request #5648: GEODE-5782: Try to improve memory reporting stats for Windows

[GitBox <gi...@apache.org>]

dschneider-pivotal commented on a change in pull request #5648:
URL: https://github.com/apache/geode/pull/5648#discussion_r512149019



##########
File path: geode-core/src/integrationTest/java/org/apache/geode/distributed/LauncherMemberMXBeanIntegrationTest.java
##########
@@ -227,6 +227,8 @@ public void showJVMMetrics_returnsOJVMMetricsType()
         if (cdsValue.equals(osMetricValue)) {
           break;
         }
+        System.gc();
+        System.runFinalization();

Review comment:
       or can you change the test to be more patient to see an expected value? Do you know how often these values are currently being updated?




----------------------------------------------------------------
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:
users@infra.apache.org