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

Extract: importers/chunk module as go-ipfs-chunker #4661

Merged
merged 6 commits into from
Feb 8, 2018

Conversation

hsanjuan
Copy link
Contributor

@hsanjuan hsanjuan commented Feb 5, 2018

This extracts chunk to https://github.com/ipfs/go-ipfs-chunker

This is part of an effort to extract importer from go-ipfs so they can be re-used by ipfs-cluster with more liberty (and without having to import go-ipfs itself).

Before I leave everything ready to merge, is the naming go-ipfs-chunker ok? It might be worth to go all the way and rename the whole thing and references to go-ipfs-splitter, or to rename the Splitter interface to Chunker.

@ghost ghost assigned hsanjuan Feb 5, 2018
@ghost ghost added the status/in-progress In progress label Feb 5, 2018
@hsanjuan hsanjuan changed the title Do not merge. WIP: Extract: importers/chunk module as go-ipfs-chunker Extract: importers/chunk module as go-ipfs-chunker Feb 7, 2018
@Stebalien
Copy link
Member

@hsanjuan please preserve git history when possible (in go-ipfs-chunker). I know we have directions for how to do this somewhere but I can't find them (@whyrusleeping?).

@whyrusleeping
Copy link
Member

License: MIT
Signed-off-by: Hector Sanjuan <[email protected]>
License: MIT
Signed-off-by: Hector Sanjuan <[email protected]>
License: MIT
Signed-off-by: Hector Sanjuan <[email protected]>
License: MIT
Signed-off-by: Hector Sanjuan <[email protected]>
@hsanjuan
Copy link
Contributor Author

hsanjuan commented Feb 8, 2018

Thanks, that was useful.

I have:

  • Imported history to go-ipfs-chunker
  • Added README
  • Rebased this PR and resolved conflicts
  • Point it to the latest go-ipfs-chunker gx release (which is pinned in mars)
  • Re-ordered the imports to be consistent and rename chunk to chunker for the same reason.

Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looked it over and everything looks right.

Copy link
Member

@whyrusleeping whyrusleeping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, gx deps look good too.

@whyrusleeping whyrusleeping merged commit 990e4df into master Feb 8, 2018
@whyrusleeping whyrusleeping deleted the extract/chunk branch February 8, 2018 16:44
@ghost ghost removed the status/in-progress In progress label Feb 8, 2018
hacdias pushed a commit to ipfs/boxo that referenced this pull request Jan 27, 2023
Extract: importers/chunk module as go-ipfs-chunker

This commit was moved from ipfs/kubo@990e4df
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants