Vanlightly opened a new pull request #2861:
URL: https://github.com/apache/bookkeeper/pull/2861


   ### Motivation
   
   See BP-44 for full motivation. 
   
   This is a draft PR, I will add commits as I go.
   
   ### Changes
   
   (so far)
   
   OrderedExecutor/Scheduler
   - Removed double nesting of TimedCallable/TimedRunnable from 
OrderedExecutor. These were being added by both the decorators and explicitly 
in the overridden methods causing double recording of metrics.
   - Moved thread ordinal numbers in metric names to a label, in line with 
modern practices.
   - Added thread-scoped metrics. Configurable, disabled by default. Only 
enabled in the BookieRequestProcessor. Not in the client or any tests.
   
   
   
   
   Master Issue: #2834
   


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to