BenjaminPerryRoss commented on a change in pull request #5391:
URL: https://github.com/apache/geode/pull/5391#discussion_r464609432



##########
File path: 
geode-core/src/main/java/org/apache/geode/internal/cache/CachePerfStats.java
##########
@@ -274,7 +278,14 @@
         "Current number of regions configured for reliablity that are missing 
required roles with Limited access";
     final String reliableRegionsMissingNoAccessDesc =
         "Current number of regions configured for reliablity that are missing 
required roles with No access";
-    final String clearsDesc = "The total number of times a clear has been done 
on this cache.";
+    final String regionClearsDesc =
+        "The total number of times a clear has been done on this cache.";
+    final String bucketClearsDesc =
+        "The total number of times a clear has been done on this region and 
it's bucket regions";
+    final String partitionedRegionClearLocalDurationDesc =
+        "The time in nanoseconds partitioned region clear has been running for 
the region on this member";

Review comment:
       I actually switched this from the original metric I used to nanoseconds 
because other stats in the file seem to be using them and I wanted to be 
consistent (such as EventQueueThrottleTime or any of the tx lifetime stats in 
that class).




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