-
Notifications
You must be signed in to change notification settings - Fork 1
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
added partial #49
added partial #49
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #49 +/- ##
===========================================
+ Coverage 41.20% 54.07% +12.86%
===========================================
Files 10 11 +1
Lines 216 307 +91
===========================================
+ Hits 89 166 +77
- Misses 127 141 +14 ☔ View full report in Codecov by Sentry. |
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.
Generally LGTM, but it would be nice to have some tests 🥲
Simplified partial_restore. Co-authored-by: Bartosz Różański <bartek.roza@gmail.com>
It's working well for me!
|
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.
Requesting changes to fix the potential panic. Other things are nice-to-haves :)
@@ -56,12 +56,23 @@ Listed below are the exit codes and what they mean: | |||
- `7` - Invalid checksum of archive. | |||
- `8` - Cannot validate archive checksum. | |||
|
|||
|
|||
# Partial quicksync |
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.
Btw, how about naming it incremental?
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.
No big preference. I used "partial" because the existing one is "full".
Co-authored-by: Bartosz Różański <bartek.roza@gmail.com>
When testing set the
QS_BASE_URL
tohttp://quicksync-testnet-downloads.spacemesh.network/quicksync-poc
for now