-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Partial retrieval ux improvements #7610
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7610 +/- ##
==========================================
- Coverage 39.60% 39.35% -0.26%
==========================================
Files 635 637 +2
Lines 67581 67928 +347
==========================================
- Hits 26767 26733 -34
- Misses 36217 36571 +354
- Partials 4597 4624 +27
Continue to review full report at Codecov.
|
cdcbf20
to
8737618
Compare
7ff43a1
to
3e70b84
Compare
d31d4d5
to
bdac11a
Compare
9b5e01b
to
135796d
Compare
f8996a5
to
210485d
Compare
node/impl/client/client.go
Outdated
|
||
if p.LastBlock.Link == nil { | ||
// this is likely the root node that we've matched here | ||
// todo: is this a correct assumption |
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.
remove
TODO:
retrieve stat
commandretrieve ls
commandretrieve cat
command