[GitHub] [bookkeeper] pkumar-singh commented on pull request #2973: checkAllLedgers in Auditor supports read throttle

[GitBox <gi...@apache.org>]

pkumar-singh commented on pull request #2973:
URL: https://github.com/apache/bookkeeper/pull/2973#issuecomment-1008108825


   But I am still not sure , how it addresses the root cause. I understand that throttling with semaphore sort of reduces the pressure on the bookie. And there is a another mechanism  for that too, that is percentageOfLedgerFragmentToBeVerified(Slight misnomer here, it actually checks percentage of entries in the ledger fragments). I understand throttling will reduce timeout from bookie. But timeout can still happen and will happen. My question is why not address this issue that occasional timeout is not considered a failure.  


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