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

Optimize block range roots computation #1655

Merged
merged 2 commits into from
Apr 26, 2024

Conversation

jpraynaud
Copy link
Member

@jpraynaud jpraynaud commented Apr 26, 2024

Content

This PR includes an optimization on the computation of the block range roots so that there is a limited accumulation of the computed root before they are stored in the database (in particular to avoid the error variable number must be between ?1 and ?32766 from SQLite).

Pre-submit checklist

  • Branch
    • Crates versions are updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Issue(s)

Relates to #1633

Flush buffer every 100 root computed to avoid insert queries with more than sqlite max allowed variables.
- 'mithril-aggregator' from '0.4.64' to '0.4.65'
- 'mithril-signer' from '0.2.128' to '0.2.129'.
@jpraynaud jpraynaud self-assigned this Apr 26, 2024
Copy link

Test Results

    3 files  ±0     43 suites  ±0   8m 15s ⏱️ -18s
  976 tests ±0    976 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 070 runs  ±0  1 070 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 4c28f6c. ± Comparison against base commit 0673352.

@jpraynaud jpraynaud merged commit b914203 into main Apr 26, 2024
41 checks passed
@jpraynaud jpraynaud deleted the jpraynaud/1633-optimize-block-range-root-computation branch April 26, 2024 15:54
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