massakam opened a new pull request, #3214:
URL: https://github.com/apache/bookkeeper/pull/3214

   ### Motivation
   
   The read throttling feature in `Auditor#checkAllLedgers` added by #2973, but 
when it was enabled `checkAllLedgers` got stuck.
   
   The cause is that, as mentioned in #3070, all the 
`BookKeeperClientWorker-OrderedExecutor` threads wait for the semaphore to be 
released, and there is no thread that can execute 
`EntryExistsCallback#readEntryComplete` where the semaphore is released.
   
   ### Changes
   
   Add a new thread pool to the `LedgerChecker` class. The 
`BookKeeperClientWorker-OrderedExecutor` threads threads delegate subsequent 
processing (this includes waiting for the semaphore to be released) to threads 
in this thread pool as soon as they release the semaphore.
   
   This will prevent the `BookKeeperClientWorker-OrderedExecutor` from getting 
stuck due to waiting for the semaphore to be released.
   
   Master Issue: #3070


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