-
Notifications
You must be signed in to change notification settings - Fork 22
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.
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
[backup] Improved writing and reading of blockchain backup file #297
[backup] Improved writing and reading of blockchain backup file #297
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## dev #297 +/- ##
==========================================
- Coverage 51.21% 47.67% -3.54%
==========================================
Files 117 131 +14
Lines 17525 19836 +2311
==========================================
+ Hits 8975 9457 +482
- Misses 7789 9561 +1772
- Partials 761 818 +57
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your great job.
I do like the producer consumer multithreading model.
Only a few questions left.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
BlockByNumber
, circumventing block range limits (grpc load balancing is now available)Changes include
Checklist
Testing
Manual tests
create backup
restore backup