You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2019/11/14 17:19:09 UTC

[GitHub] [accumulo-testing] milleruntime commented on issue #118: Adding Accumulo Availability Monitor

milleruntime commented on issue #118: Adding Accumulo Availability Monitor
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-553989168
 
 
   Hey @tushardhadiwal thanks for the contribution!   At first glance, this code seems like it could be a useful scan timing measurement tool.  FYI Accumulo already has a general purpose availability monitor (named Monitor) in the [main repo](https://accumulo.apache.org/docs/2.x/administration/monitoring-metrics).  My only initial comment is that a different name, like "ScanProbe" or "ScanTimer", would be better to avoid confusion with the Monitor. 
   

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