[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493961781 ## File path: geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java ## @@ -39,69 +39,71 @@ * @see org.apache.g

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493896136 ## File path: geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java ## @@ -39,69 +39,71 @@ * @see org.apache.g

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493896436 ## File path: geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java ## @@ -39,69 +39,71 @@ * @see org.apache.g

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493822296 ## File path: geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java ## @@ -39,69 +39,71 @@ * @see org.apache.g

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493821071 ## File path: geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java ## @@ -39,69 +39,71 @@ * @see org.apache.g

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493153715 ## File path: geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java ## @@ -339,6 +342,41 @@ public void testRegionCo

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493153715 ## File path: geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java ## @@ -339,6 +342,41 @@ public void testRegionCo