Skip to content

Conversation

@szetszwo
Copy link
Contributor

@szetszwo szetszwo requested a review from adoroszlai January 22, 2024 21:15
@adoroszlai
Copy link
Contributor

Thanks @szetszwo for the patch, makes sense to me. (16 attempts for 5 retries looked like magic.)

@szetszwo szetszwo merged commit c1e0ec2 into apache:master Jan 22, 2024
@szetszwo
Copy link
Contributor Author

16 attempts for 5 retries looked like magic.

Yes, it is magic. I am not sure why it seems to have a fixed 16 attempts with the bug.

Thanks a lot for reviewing this!

SzyWilliam pushed a commit that referenced this pull request Mar 27, 2024
symious pushed a commit to symious/ratis that referenced this pull request Apr 4, 2024
SzyWilliam pushed a commit to SzyWilliam/ratis that referenced this pull request Jun 12, 2024
szetszwo added a commit to szetszwo/ratis that referenced this pull request Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants