Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Analyse and consider use of retryUpToAndContinue instead of retryUpTo #1126

Open
HonzaR opened this issue Jul 20, 2023 · 0 comments
Open

Analyse and consider use of retryUpToAndContinue instead of retryUpTo #1126

HonzaR opened this issue Jul 20, 2023 · 0 comments
Assignees
@HonzaR
Labels
enhancement New feature or request

Comments

@HonzaR
Copy link
Collaborator

HonzaR commented Jul 20, 2023

Describe the issue

We'd like to check and consider the use of the retryUpToAndContinue function instead of retryUpTo in several occurrences in CompactblockProcessor, as the second one is terminating the general sync blocks retry loop.

Any extra information

We already have the retryUpToAndContinue function added as part of the latest change sin #1116.
@HonzaR HonzaR added the enhancement New feature or request label Jul 20, 2023
@HonzaR HonzaR self-assigned this Jul 20, 2023
@str4d str4d added this to the Zashi 1.0 performance milestone Jan 11, 2024
@str4d str4d removed this from the Zashi 1.0 performance milestone Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HonzaR HonzaR

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@str4d @HonzaR