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

fault_proving(global_roots): scaffold global merkle root storage crate #2553

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

netrome
Copy link
Contributor

@netrome netrome commented Jan 13, 2025

Closes #2552

This PR adds a new crate with column definitions for the global state root service, which we intend to deploy as a standalone binary.

Moreover, this crate also contains a top-level function to populate these columns with data given a new block.

@netrome netrome linked an issue Jan 13, 2025 that may be closed by this pull request
@netrome netrome requested review from rymnc and a team January 13, 2025 14:16
crates/storage/src/column.rs Outdated Show resolved Hide resolved
crates/storage/src/structured_storage/coins.rs Outdated Show resolved Hide resolved
@netrome netrome changed the title feat: feature flag for global state roots + coins table implementation feat: scaffold merkle root storage implementation Jan 16, 2025
@netrome
Copy link
Contributor Author

netrome commented Jan 16, 2025

Redefined the PR and issue after @xgreenx's proposal. I will do some further cleanups and create follow-up issues for the next steps, before marking this PR as ready for review.

@netrome netrome changed the title feat: scaffold merkle root storage implementation fault_proving(global_roots): scaffold merkle root storage crate Jan 16, 2025
@netrome netrome force-pushed the 2552-feature-flag-for-global-state-roots-+-first-onchain-table-implementation branch from 5935b3d to f25d6d0 Compare January 16, 2025 13:51
@netrome netrome force-pushed the 2552-feature-flag-for-global-state-roots-+-first-onchain-table-implementation branch from 20c4ac8 to b3a2b04 Compare January 16, 2025 14:39
@netrome netrome force-pushed the 2552-feature-flag-for-global-state-roots-+-first-onchain-table-implementation branch from 524d43f to cff896d Compare January 16, 2025 15:07
@netrome netrome force-pushed the 2552-feature-flag-for-global-state-roots-+-first-onchain-table-implementation branch from 99ad6a9 to cf3a62b Compare January 16, 2025 16:17
@netrome netrome marked this pull request as ready for review January 16, 2025 20:50
@netrome netrome requested review from a team, acerone85 and xgreenx January 16, 2025 20:51
@netrome netrome changed the title fault_proving(global_roots): scaffold merkle root storage crate fault_proving(global_roots): scaffold global merkle root storage crate Jan 16, 2025
Copy link
Collaborator

@xgreenx xgreenx left a comment

Choose a reason for hiding this comment

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

It would be nice to verify that new crate us not std compatible by updating our CI and ./ci_check.sh script=)

@netrome
Copy link
Contributor Author

netrome commented Jan 17, 2025

It would be nice to verify that new crate us not std compatible by updating our CI and ./ci_check.sh script=)

Good point, fixed the no-std build and updated the CI checks in e0bd85c and 1c1404d

@netrome netrome requested a review from xgreenx January 17, 2025 09:42
ci_checks.sh Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fault_proving(global_roots): Scaffold merkle root storage crate
4 participants