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 2020/10/29 18:56:07 UTC
[GitHub] [bookkeeper] karanmehta93 opened a new pull request #2470: Introduce new retry policy for metadata operations
karanmehta93 opened a new pull request #2470:
URL: https://github.com/apache/bookkeeper/pull/2470
Descriptions of the changes in this PR:
### Motivation
Current retry backoff policy is truly exponential, hence retries are spaced out at large intervals. There is maxBackOff time that limits it, but it doesn't have a concept of deadline. Hence, applications with shorter SLA's have issues dealing with ZK failures and recovering from it.
### Changes
The new policy introduces an aggressive retry mechanism with the backoff constants as less exponential. The deadline ensures that we retry at least once at the deadline. The stack trace would indicate the operation at risk.
This doesn't update the product code, which is still in testing. I am splitting PR's for for sake of reviews.
----------------------------------------------------------------
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] jiazhai merged pull request #2470: Introduce new retry policy for metadata operations
Posted by GitBox <gi...@apache.org>.
jiazhai merged pull request #2470:
URL: https://github.com/apache/bookkeeper/pull/2470
----------------------------------------------------------------
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 commented on pull request #2470: Introduce new retry policy for metadata operations
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #2470:
URL: https://github.com/apache/bookkeeper/pull/2470#issuecomment-720289340
@Ghatage this patch is not trivial, I guess you "approve" it.
feel free to merge
----------------------------------------------------------------
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] Ghatage commented on pull request #2470: Introduce new retry policy for metadata operations
Posted by GitBox <gi...@apache.org>.
Ghatage commented on pull request #2470:
URL: https://github.com/apache/bookkeeper/pull/2470#issuecomment-719017582
rerun failure checks
----------------------------------------------------------------
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