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

Publish Pando-compatible metadata over GraphSync #372

Merged
merged 1 commit into from
Feb 25, 2022

Conversation

masih
Copy link
Member

@masih masih commented Feb 25, 2022

Implement a publisher specific to Pando metadata that uses the CID of
RecordUpdate as byte value of Pando metadata and exposes a GraphSync
server that can resolve both Pando metadata, and the RecordUpdate
CIDs.

Decouple statedb from publication by changing the state interface to
return the CID of RecordUpdate. This removes all libp2p and legs code
from the state package. Instead, the publication is done on the
controller itself upon POSTs to /complete/<workedby> as a
non-critical operation. This makes a better structured code in the
context of Separation of Concerns.

Relates to:

@masih masih force-pushed the masih/pando-metadata-struct branch from 570cde7 to 0663ac7 Compare February 25, 2022 12:12
Implement a publisher specific to Pando metadata that uses the CID of
`RecordUpdate` as byte value of Pando metadata and exposes a GraphSync
server that can resolve both Pando metadata, and the `RecordUpdate`
CIDs.

Decouple `statedb` from publication by changing the state interface to
return the CID of `RecordUpdate`. This removes all libp2p and legs code
from the `state` package. Instead, the publication is done on the
controller itself upon `POST`s to `/complete/<workedby>` as a
non-critical operation. This makes a better structured code in the
context of Separation of Concerns.

Relates to:
 - #342
@masih masih force-pushed the masih/pando-metadata-struct branch from 0663ac7 to a08d090 Compare February 25, 2022 12:20
@masih masih marked this pull request as ready for review February 25, 2022 12:30
@masih masih merged commit b57f2e2 into main Feb 25, 2022
@masih masih deleted the masih/pando-metadata-struct branch February 25, 2022 13:02
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.

2 participants