ThreadPool metrics bound to default global registry, may not be getting reported

[dlmarion]

ThreadPool metrics are bound to the global registry here. I'm not sure if these meters will get published via the pluggable MeterRegistry implementations or not. Need to test for the presence of metrics with names starting with executor. to determine if they are or are not.

[ddanielr]

Was looking into some other metric issues earlier and happened to see the executor metrics with 2.1.4.

Started a local 2.1.4 accumulo cluster with general.micrometer.enabled=true which uses the LoggingMeterRegistryFactory by default.

Checked the compactor logs and saw the following metrics:

executor.completed{host=localhost,instance.name=uno,name=accumulo.pool.general.server,port=9134,process.name=compactor} throughput=0.016667 tasks/s
executor.active{host=localhost,instance.name=uno,name=accumulo.poolcompactor.client,port=9134,process.name=compactor} value=0 threads
executor.active{host=localhost,instance.name=uno,name=accumulo.pool.general.server,port=9134,process.name=compactor} value=0 threads
executor.pool.core{host=localhost,instance.name=uno,name=accumulo.pool.general.server,port=9134,process.name=compactor} value=1 threads
executor.pool.core{host=localhost,instance.name=uno,name=accumulo.poolcompactor.client,port=9134,process.name=compactor} value=1 threads
executor.pool.max{host=localhost,instance.name=uno,name=accumulo.poolcompactor.client,port=9134,process.name=compactor} value=1 threads
executor.pool.max{host=localhost,instance.name=uno,name=accumulo.pool.general.server,port=9134,process.name=compactor} value=2147483647 threads
executor.pool.size{host=localhost,instance.name=uno,name=accumulo.pool.general.server,port=9134,process.name=compactor} value=1 threads
executor.pool.size{host=localhost,instance.name=uno,name=accumulo.poolcompactor.client,port=9134,process.name=compactor} value=0 threads
executor.queue.remaining{host=localhost,instance.name=uno,name=accumulo.pool.general.server,port=9134,process.name=compactor} value=2147483647 tasks
executor.queue.remaining{host=localhost,instance.name=uno,name=accumulo.poolcompactor.client,port=9134,process.name=compactor} value=2147483647 tasks
executor.queued{host=localhost,instance.name=uno,name=accumulo.poolcompactor.client,port=9134,process.name=compactor} value=0 tasks
executor.queued{host=localhost,instance.name=uno,name=accumulo.pool.general.server,port=9134,process.name=compactor} value=1 tasks

Do you want to add a specific test for the executor prefix?