Skip to content

feat: remove compression from engine responses#9667

Merged
Gabriel-Trintinalia merged 7 commits intobesu-eth:mainfrom
Gabriel-Trintinalia:test-getblobsv2-no-compression
Jan 22, 2026
Merged

feat: remove compression from engine responses#9667
Gabriel-Trintinalia merged 7 commits intobesu-eth:mainfrom
Gabriel-Trintinalia:test-getblobsv2-no-compression

Conversation

@Gabriel-Trintinalia
Copy link
Copy Markdown
Contributor

@Gabriel-Trintinalia Gabriel-Trintinalia commented Jan 21, 2026

PR description

Improve performance for getBlobsV2

Test results #9667 (comment)

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
@siladu
Copy link
Copy Markdown
Contributor

siladu commented Jan 22, 2026

Test Results

Mainnet synced with nimbus, prysm and lighthouse

no-compression-mainnet-sync

Compared control vs test teku and saw no significant impact on newPayload or FCU

(test was on "-snooper" vm)
no-compression-engine-api-control-vs-test

Benchmark

Using random blob data and curl --compressed (to request compression) shows significant improvement across the full blob range 1 - 128.
Also shows impact of compression if blob data is all zeros (best case).
No compression > best case with compression.

blobs_performance

Improvement per blob count

blobs_performance_improvement
Blobs Compression (ms) No Compression (ms) Speedup Improvement
21 171.2 33.4 5.1x 80.5%
32 249.7 32.8 7.6x 86.9%
48 361.5 44.1 8.2x 87.8%
72 520.3 51.7 10.1x 90.1%

Copy link
Copy Markdown
Contributor

@siladu siladu left a comment

Choose a reason for hiding this comment

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

Can you add changelog to mention up to 10x improvement for large numbers of blobs please

@macfarla macfarla moved this to Open PRs in 26.1.0 Release Jan 22, 2026
@macfarla macfarla moved this from Open PRs to PFI in 26.1.0 Release Jan 22, 2026
ahamlat
ahamlat approved these changes Jan 22, 2026
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
@Gabriel-Trintinalia Gabriel-Trintinalia enabled auto-merge (squash) January 22, 2026 22:50
@Gabriel-Trintinalia Gabriel-Trintinalia merged commit 2c60d51 into besu-eth:main Jan 22, 2026
46 checks passed
@github-project-automation github-project-automation bot moved this from PFI to Done in 26.1.0 Release Jan 22, 2026
macfarla pushed a commit to CPerezz/besu that referenced this pull request Feb 6, 2026
* test uncompressed get blobs v2

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>

* comment

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>

* changelog

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>

---------

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
Co-authored-by: Simon Dudley <simon.dudley@consensys.net>
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.

4 participants