You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/05/05 03:11:38 UTC

[GitHub] [bookkeeper] gaozhangmin commented on pull request #3214: Issue 3070: Fix bug where checkAllLedgers gets stuck when read throttling is enabled

gaozhangmin commented on PR #3214:
URL: https://github.com/apache/bookkeeper/pull/3214#issuecomment-1118123312

   > @dlg99 Changed the implementation to achieve throttling using `RateLimiter` instead of `Semaphore`. What do you think?
   > 
   > I think this is simpler than the implementation of adding a new flag and requesting/releasing a permit according to its value.
   
   +1


-- 
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: issues-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org