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

Store ZQ1 blocks during sync #2287

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Store ZQ1 blocks during sync #2287

wants to merge 9 commits into from

Conversation

shawn-zil
Copy link
Contributor

@shawn-zil shawn-zil commented Feb 4, 2025

A small change that just writes the block to the database, without execution, if it contains any ZQ1 transactions.

This PR is dependent on #2054 and should use the same check.

Copy link
Contributor

github-actions bot commented Feb 4, 2025

🐰 Bencher Report

Branch2232-store-zq1-blocks
Testbedself-hosted
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
milliseconds (ms)
(Result Δ%)
Upper Boundary
milliseconds (ms)
(Limit %)
full-blocks-erc20-transfers/full-blocks-erc20-transfers📈 view plot
🚷 view threshold
1,034.40
(-16.63%)
1,634.99
(63.27%)
full-blocks-evm-transfers/full-blocks-evm-transfers📈 view plot
🚷 view threshold
445.77
(-10.96%)
719.26
(61.98%)
full-blocks-zil-transfers/full-blocks-zil-transfers📈 view plot
🚷 view threshold
4,147.10
(+5.45%)
5,142.99
(80.64%)
process-empty/process-empty📈 view plot
🚷 view threshold
10.39
(+14.20%)
11.43
(90.95%)
🐰 View full continuous benchmarking report in Bencher

@shawn-zil shawn-zil marked this pull request as ready for review February 4, 2025 07:32
@shawn-zil shawn-zil force-pushed the 2232-store-zq1-blocks branch from a8c48a8 to 49ae741 Compare February 4, 2025 08:29
zilliqa/src/node.rs Outdated Show resolved Hide resolved
@shawn-zil shawn-zil enabled auto-merge February 4, 2025 14:10
@shawn-zil shawn-zil force-pushed the 2232-store-zq1-blocks branch from 49ae741 to c51a58a Compare February 5, 2025 01:37
@shawn-zil shawn-zil linked an issue Feb 5, 2025 that may be closed by this pull request
@shawn-zil shawn-zil requested a review from bzawisto February 5, 2025 03:58
@shawn-zil shawn-zil marked this pull request as draft February 5, 2025 10:55
auto-merge was automatically disabled February 5, 2025 10:55

Pull request was converted to draft

@shawn-zil shawn-zil requested a review from 86667 February 6, 2025 01:10
@shawn-zil shawn-zil marked this pull request as ready for review February 6, 2025 06:07
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.

When syncing, just store ZQ1 blocks
2 participants