[GitHub] [bookkeeper] rdhabalia opened a new pull request #2336: [bookkeeper-tool] Fix update rate-limiting for update-ledger command

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

rdhabalia opened a new pull request #2336:
URL: https://github.com/apache/bookkeeper/pull/2336


   ### Motivation
   Right now, `UpdateLedgerCmd` provides option `updatespersec` to throttle number of writes on zk. However, it throttles number of reads instead writes. Because of that it takes long time to complete this command as it slows down the read instead applying throttling while writing to zk.
   
   ### Modification
   - Apply throttling while updating zk.
   - If writes are being throttled then we also want to avoid accumulating reads so, added `maxOutstandingReads` option to manage max concurrent reads.
   
   **Note:** We also need this change for #2321 .. so, I will rebase #2321 once this PR is merged.


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

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

[GitHub] [bookkeeper] eolivelli merged pull request #2336: [bookkeeper-tool] Fix update rate-limiting for update-ledger command

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

eolivelli merged pull request #2336:
URL: https://github.com/apache/bookkeeper/pull/2336


   


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

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