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