You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by dh...@apache.org on 2019/07/22 22:09:23 UTC

[geode] 10/18: Add TODO statements

This is an automated email from the ASF dual-hosted git repository.

dhemery pushed a commit to branch GEODE-7001-region-entry-count
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 88c8612e3627448d94e022bc24b1886366dfc9dd
Author: Kirk Lund <kl...@apache.org>
AuthorDate: Thu Jul 18 09:32:15 2019 -0700

    Add TODO statements
---
 .../java/org/apache/geode/metrics/RegionEntriesGaugeTest.java           | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/geode-assembly/src/acceptanceTest/java/org/apache/geode/metrics/RegionEntriesGaugeTest.java b/geode-assembly/src/acceptanceTest/java/org/apache/geode/metrics/RegionEntriesGaugeTest.java
index 28b2dac..00dac99 100644
--- a/geode-assembly/src/acceptanceTest/java/org/apache/geode/metrics/RegionEntriesGaugeTest.java
+++ b/geode-assembly/src/acceptanceTest/java/org/apache/geode/metrics/RegionEntriesGaugeTest.java
@@ -54,6 +54,7 @@ public class RegionEntriesGaugeTest {
   @Rule
   public TemporaryFolder temporaryFolder = new TemporaryFolder();
 
+  // TODO: refactor to JarRule
   @Rule
   public MetricsPublishingServiceJarRule metricsPublishingServiceJarRule =
       new MetricsPublishingServiceJarRule("metrics-publishing-service.jar",
@@ -144,6 +145,7 @@ public class RegionEntriesGaugeTest {
         stopLocatorCommand);
   }
 
+  // TODO: add 2nd region to verify we are reading entries for just one
   @Test
   public void regionEntriesGaugeShowsCountOfReplicateRegionValuesInServer() {
     String regionName = "replicateRegion";