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

Problem: UTXO implementation unfinished for release. #2132

Merged
merged 1 commit into from
Mar 14, 2018

Conversation

kansi
Copy link
Contributor

@kansi kansi commented Mar 14, 2018

The current UTXO implementation requires some more work in order for it to be production ready. So, it is better to rollback the feature for time being for the alpha release.

Solution: Rollback UTXO for the alpha release.
@kansi kansi requested review from vrde and ttmc March 14, 2018 07:09
@ttmc
Copy link
Contributor

ttmc commented Mar 14, 2018

I wondered about this.

Let's get it right, e.g. write a BEP, understand how we store the UTXO data, and the app hash, make sure Sylvain is on board, maybe use some of the tools from the Tendermint team for storing/manipulating IAVL+ trees, etc.

Can you write a bit about what exists now? Maybe in a short "Informational" BEP? Don't spend too long on that, but it would be useful for context for everyone. We can then reference it when we write a long "Standard Track" BEP to describe its replacement.

👍 from me.

@ttmc
Copy link
Contributor

ttmc commented Mar 14, 2018

I will bump the bullet point about UTXO and app hash to the next release in the ROADMAP.md document.

@kansi kansi requested a review from sbellem March 14, 2018 09:34
@kansi
Copy link
Contributor Author

kansi commented Mar 14, 2018

@ttmc That sounds good. I will write an information BEP to the best of my understanding.

@kansi
Copy link
Contributor Author

kansi commented Mar 14, 2018

@ttmc Please refer bigchaindb/BEPs#20

@ttmc
Copy link
Contributor

ttmc commented Mar 14, 2018

I made some comments on bigchaindb/BEPs#20

Copy link
Contributor

@vrde vrde left a comment

Choose a reason for hiding this comment

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

Thanks @kansi 👏

@vrde vrde merged commit 8c32ae7 into bigchaindb:tendermint Mar 14, 2018
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.

None yet

3 participants