application-research / autoretrieve

A server to make GraphSync data accessible on IPFS
22 stars 7 forks source link

feat: use lassie for retrievals #149

Closed rvagg closed 1 year ago

rvagg commented 1 year ago

Not intended to be merged (yet anyway, we can have that conversation later). This is for dev purposes to make sure we get interfaces right and give me something functional so I can actually use code over in lassie for something real.

Depends on:

codecov-commenter commented 1 year ago

Codecov Report

Base: 20.90% // Head: 5.70% // Decreases project coverage by -15.20% :warning:

Coverage data is based on head (0a575d2) compared to base (f1df860). Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #149 +/- ## ========================================== - Coverage 20.90% 5.70% -15.21% ========================================== Files 16 10 -6 Lines 2334 1490 -844 ========================================== - Hits 488 85 -403 + Misses 1816 1401 -415 + Partials 30 4 -26 ``` | [Impacted Files](https://codecov.io/gh/application-research/autoretrieve/pull/149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research) | Coverage Δ | | |---|---|---| | [autoretrieve.go](https://codecov.io/gh/application-research/autoretrieve/pull/149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-YXV0b3JldHJpZXZlLmdv) | `0.00% <0.00%> (ø)` | | | [bitswap/provider.go](https://codecov.io/gh/application-research/autoretrieve/pull/149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-Yml0c3dhcC9wcm92aWRlci5nbw==) | `0.00% <0.00%> (ø)` | | | [config.go](https://codecov.io/gh/application-research/autoretrieve/pull/149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-Y29uZmlnLmdv) | `0.00% <0.00%> (ø)` | | | [endpoint/estuary.go](https://codecov.io/gh/application-research/autoretrieve/pull/149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-ZW5kcG9pbnQvZXN0dWFyeS5nbw==) | `0.00% <0.00%> (ø)` | | | [endpoint/indexer.go](https://codecov.io/gh/application-research/autoretrieve/pull/149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-ZW5kcG9pbnQvaW5kZXhlci5nbw==) | `0.00% <0.00%> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

hannahhoward commented 1 year ago

@rvagg @kylehuntsman 's PR is merged can we move this forward? Any reason not to?

rvagg commented 1 year ago

https://github.com/application-research/filclient/pull/112 is the blocker, and it's blocked by having a tagged boost—we could just use an untagged master over there but I don't think we need to rush this, I'm happy to keep this branch updated on both ends for now