fanjianye opened a new pull request #3037:
URL: https://github.com/apache/bookkeeper/pull/3037


   Descriptions of the changes in this PR:
   
   
   
   ### Motivation
   
   1. the code style in BookieStats is different to other Stats
   2. the BOOKIE_READ_ENTRY_BYTES metric in BookieStats is not correct 
   
   ### Changes
   
   1. fix the code style problem in BookieStats
   2. since the BOOKIE_READ_ENTRY_BYTES and the READ_BYTES metric both collect 
entry.readableBytes() in BookieImpl#readEntry(),it is better to keep the 
counter, not OpStatsLogger. OpStatsLogger is relatively expensive.
   
   


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