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

tupelo:next #344

Closed
tobowers opened this issue Sep 13, 2019 · 7 comments
Closed

tupelo:next #344

tobowers opened this issue Sep 13, 2019 · 7 comments

Comments

@tobowers
Copy link
Contributor

tobowers commented Sep 13, 2019

We have a breaking data change coming for tupelo in form of supporting Conditions. Other breaking changes I'd like to explore:

  • transactionID on the CurrentState (to become TreeState) to be the CID of the cbor serialized AddBlockRequest
  • support conditions (CurrentState => TreeState, etc)
    ** perhaps add a version field to TreeState
  • EXPLORE: keep tupelo currentState in a hamt

Hamt:

@zonotope
Copy link
Contributor

Another one: moving the Token and related structs to protobufs in the messages repo

@tobowers
Copy link
Contributor Author

tobowers commented Sep 13, 2019

oh and adding custom tags to the generated protobufs to omit empty in cbor

@brandonwestcott
Copy link
Contributor

adding the chain height index

@tobowers
Copy link
Contributor Author

adding the chain height index

at which point we should consider "auto indexes"

@zonotope
Copy link
Contributor

Would the plan here be to work out the other compatibility issues (including the slight error mismatch of your open pr + the cid change you mentioned) and then work towards pushing those changes upstream to eventually use upstream instead of the fork, or do you think we'd still maintain the fork going forward?

@tobowers
Copy link
Contributor Author

tobowers commented Sep 13, 2019 via email

@tobowers
Copy link
Contributor Author

this is pretty out of date by this point.

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

No branches or pull requests

3 participants