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.

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

Wip range download #2015

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Wip range download #2015

wants to merge 6 commits into from

Conversation

rklaehn
Copy link
Contributor

@rklaehn rklaehn commented Feb 9, 2024

Description

Add some more ops related to range requests to iroh for a demo

Notes & open questions

  1. This is far from being ready to merge in terms of quality
  2. We need to make sure the blob api does not become a giant list of stuff somebody at some point found useful, but a consistent API.

Change checklist

  • Self-review.
  • Documentation updates if relevant.
  • Tests if relevant.

@rklaehn
Copy link
Contributor Author

rklaehn commented Feb 9, 2024

First part:

❯ cargo run blob local-ranges --hash 17c8e5a70f69d289105553dfa762064e8cc57ffb9355d76508f557e37ef051db
0..97657

❯ printf "\x00" | dd bs=1 seek=100000 count=1 of=$APP_HOME/iroh/blobs.v1/complete/17c8e5a70f69d289105553dfa762064e8cc57ffb9355d76508f557e37ef051db.data conv=notrunc 

❯ cargo run blob local-ranges --hash 17c8e5a70f69d289105553dfa762064e8cc57ffb9355d76508f557e37ef051db     
0..96
112..97657

these are chunk ranges, so writing 0x00 at offset 100000 damaged chunk 97, so chunk group 96..112.

@rklaehn
Copy link
Contributor Author

rklaehn commented Feb 9, 2024

Second part:

cargo run -- blob download-ranges --bytes 1..100 <ticket>

@dignifiedquire dignifiedquire added this to the v0.14.0 milestone Feb 12, 2024
@dignifiedquire dignifiedquire removed this from the v0.14.0 milestone Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📋 Backlog
Development

Successfully merging this pull request may close these issues.

None yet

2 participants