[GitHub] [accumulo] ctubbsii commented on a change in pull request #1376: Add gc metrics reporting

[GitBox <gi...@apache.org>]

ctubbsii commented on a change in pull request #1376: Add gc metrics reporting
URL: https://github.com/apache/accumulo/pull/1376#discussion_r328808994
 
 

 ##########
 File path: core/src/main/java/org/apache/accumulo/core/conf/Property.java
 ##########
 @@ -561,6 +561,8 @@
           + " to force the changes to be written to disk, the metadata and root tables can be flushed"
           + " and possibly compacted. Legal values are: compact - which both flushes and compacts the"
           + " metadata; flush - which flushes only (compactions may be triggered if required); or none"),
+  GC_ENABLE_METRICS2("gc.enable.metrics2", "true", PropertyType.BOOLEAN,
 
 Review comment:
   Should these be enabled by default when they weren't before?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services