ipfs / boxo

A set of reference libraries for building IPFS applications and implementations in Go.
https://github.com/ipfs/boxo#readme
Other
202 stars 87 forks source link

New Unixfs implementations #224

Open Jorropo opened 1 year ago

Jorropo commented 1 year ago

Dedicated unixfs implementations are often much more performant and efficient. See for example two of mines:

I think it is more sustainable to maintain a handful of dedicated implementations that individually do one job well (something like feather and something that would maintain state to support unordered dags are really different).

Unified middle layer

The way I have been writing my implementations is by jugling bytes unixfs.proto and dag-pb.proto by hand everywhere. This is kinda meh looking because of all the little details that have to be checked.

I really think we need some efficient middle layer that takes in blocks.Block parse them, do sanity checks and return some representation that looks like internal unixfs (so one node per block, and full support for all schenanigans you would want to do but with error checking):

package unixfs

type Type uint8

const (
  _ Type = iota
  Directory
  File
  Metadata
  Symlink
  HAMTShard
)

type Node struct{
  Type Type
  Data []byte // for files
  Entries []Entry
  DagSize uint64
  HashType uint64 // for HAMT dirs
  Fanout uint64 // for HAMT dirs
}

type Entry struct{
  Link
  Name string // for directories
}

type Link struct{
  Type Type // Indicative, not actually
  Cid cid.Cid
  FileSize uint64 // for files
  DagSize uint64
}

func (*Node) UnmarshalIPFS(blocks.Block) error
func (*Node) MarshalIPFS() (blocks.Block, error)

func (*Node) Encode() (Link, blocks.Block, error)

// File operations
func (*Node) ParseFile(Link, blocks.Block) error
func (*Node) AddFileSegments(Link...) error
func (*Node) FileSize() uint64

// Directory operations
func (*Node) ParseDirectory(Link, blocks.Block) error
func (*Node) AddChildrenNode(Link...) error

// other types ...

This would not includes helper functions for more advanced stuff like HAMT, chunking, ... we will most likely need thoses too but the goal here is to provide a thin wrapper around the protobuf that add the repetitive input validation.

Impls

List of implementations we need:

BigLep commented 11 months ago

2023-10-05 conversation:

m0ar commented 9 months ago

@Jorropo something we'd very much like is a fast recursive ls to help figure out the structure of an unixFS DAG, basically tree. Right now, we do O(dirs) ipfs-unixfs.ls requests to a kubo node to be able to figure out a file structure, without actually fetching the files themselves. This is of course ridiculously inefficient and take ~60 seconds even with our own node within AWS. We can't get the entire DAG, including the leaves, because it's generally way too large.

So, my questions:

  1. Is this recursive ls something what would be in scope for boxo, or is it the wrong abstraction layer?
  2. Could I use boxo to implement a stand-alone service to do it?
  3. Is it better to implement it inside kubo, when this boxo improvement gets delivered?