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

fix: private trie cache db with gc #1235

Merged
merged 12 commits into from
Aug 31, 2021
Merged

fix: private trie cache db with gc #1235

merged 12 commits into from
Aug 31, 2021

Conversation

baptiste-b-pegasys
Copy link
Contributor

@baptiste-b-pegasys baptiste-b-pegasys commented Aug 11, 2021

Don't save the private trie DB at every commit.

Use the same way as the public trie DB, where data stored are optimized with cache and GC.

@baptiste-b-pegasys baptiste-b-pegasys self-assigned this Aug 11, 2021
@baptiste-b-pegasys baptiste-b-pegasys marked this pull request as ready for review August 17, 2021 13:38
ricardolyn
ricardolyn previously approved these changes Aug 30, 2021
Copy link
Contributor

@ricardolyn ricardolyn left a comment

Choose a reason for hiding this comment

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

Looks good. only missing comments

core/blockchain.go Show resolved Hide resolved
core/blockchain.go Show resolved Hide resolved
core/blockchain.go Show resolved Hide resolved
core/blockchain.go Show resolved Hide resolved
core/blockchain.go Show resolved Hide resolved
core/blockchain.go Show resolved Hide resolved
core/blockchain.go Show resolved Hide resolved
core/blockchain.go Outdated Show resolved Hide resolved
core/blockchain.go Outdated Show resolved Hide resolved
core/blockchain.go Outdated Show resolved Hide resolved
Copy link
Contributor

@nmvalera nmvalera left a comment

Choose a reason for hiding this comment

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

LGTM

@ricardolyn ricardolyn merged commit 614b212 into master Aug 31, 2021
@ricardolyn ricardolyn deleted the fix/private-trie branch August 31, 2021 09:51
danielporto pushed a commit to danielporto/quorum that referenced this pull request Sep 7, 2021
* fix: private trie cache db with gc

* fix: commit

* fix: use the right private root

* contract implementation enforcement

* fix merge

* GoQ comments

* Apply suggestions from code review

GoQuorum Comments
baptiste-b-pegasys added a commit that referenced this pull request Oct 1, 2021
Bug found in the MPS, want to package this feature for both dual state and MPS.

This reverts commit 614b212.
ricardolyn pushed a commit that referenced this pull request Oct 1, 2021
Bug found in the MPS, want to package this feature for both dual state and MPS.

This reverts commit 614b212.
@baptiste-b-pegasys baptiste-b-pegasys restored the fix/private-trie branch January 24, 2022 07:19
@baptiste-b-pegasys baptiste-b-pegasys deleted the fix/private-trie branch May 19, 2022 07:46
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.

3 participants