dschneider-pivotal commented on a change in pull request #7456:
URL: https://github.com/apache/geode/pull/7456#discussion_r832733268
##########
File path:
geode-core/src/integrationTest/java/org/apache/geode/internal/statistics/GemFireStatSamplerIntegrationTest.java
##########
@@ -497,37 +492,6 @@ public void testArchiveRemoval() throws Exception {
waitForFileToDelete(archiveFile1, 10 * sampleRate, 10);
}
- @Test
- public void testLocalStatListenerRegistration() throws Exception {
Review comment:
The product no longer registers a stat listener for heap pool
monitoring. This test was testing it. This test already has other tests that
verify the stat listener feature in general (with no reference to the heap pool
monitoring). So the only coverage we are losing is for code that no longer
exists.
--
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.
To unsubscribe, e-mail: notifications-unsubscr...@geode.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
