search

filecoin-project / venus

Filecoin Full Node Implementation in Go
https://venus.filecoin.io
Other
2.06k stars 459 forks source link

Remove Bitswap Fetcher once Graphsync Fetcher is battle tested #3191

Closed hannahhoward closed 5 years ago

hannahhoward commented 5 years ago

Description

3190 makes Graphsync the default fetcher for Blockchains in Filecoin. Once this goes in production in 0.4, and any issues are worked out, there no need to keep the Bitswap Fetcher around, and the code can be removed.

Acceptance criteria

Bitswap Fetcher is not in Filecoin codebase

Risks + pitfalls

Again, Graphsync Fetcher is not battle tested yet, so this shouldn't be done until we're sure we won't need the Bitswap Fetcher

Where to begin

n/a

anorth commented 5 years ago

This is a good first issue after 9 September 2019.