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

'ipfs dag patch' #4782

Open
whyrusleeping opened this issue Mar 7, 2018 · 3 comments
Open

'ipfs dag patch' #4782

whyrusleeping opened this issue Mar 7, 2018 · 3 comments
Assignees
Labels
kind/enhancement A net-new feature or improvement to an existing feature

Comments

@whyrusleeping
Copy link
Member

I recently chatted with a user who is using ipfs object patch to maintain a KV storage tree in ipfs. We should look into adding a patch command to ipfs dag so that users of object patch can migrate.

Note that this isnt quite as simple to implement as ipfs object patch is, but should still be doable under certain constraints.

@whyrusleeping whyrusleeping added the kind/enhancement A net-new feature or improvement to an existing feature label Mar 7, 2018
@Stebalien
Copy link
Member

Personally, I'd use (something like) JSON Patch. We could even use IPLD objects as patches.

> ipfs dag patch apply QmObj QmPatch1 QmPatch2...
> ipfs dag patch apply QmId < patch.json
> ipfs dag patch add QmObj /path "value"
...

However, we'll need a general purpose dag encoder system to get this working properly (we currently only have a general purpose dag decoder system).

@schomatis
Copy link
Contributor

Waiting to resolve #4801 before tackling this since many of the decisions there will also apply here.

@BigLep BigLep added this to the Best Effort Track milestone Mar 10, 2022
@lidel
Copy link
Member

lidel commented Jun 20, 2022

Relevant IPLD spec: https://ipld.io/specs/patch/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature
Projects
No open projects
Status: 🛑 Blocked
Development

No branches or pull requests

5 participants