Skip to content

Conversation

@siladu
Copy link
Contributor

@siladu siladu commented Nov 17, 2025

Building on #9426.

Use HexUtils.toFastHex for engine_getPayload related requests including engine_getPayloadBodies*

Main change is 84e3fdf (#9445)

Also use parallelStreams 8c4f518 (#9445)

The most impactful optimisation is for BlobsBundleV2 used in engine_getPayloadV5+ as it is impacted by blob scaling the same way as #9426

Tested engine_getPayloadV5 impact on a hoodi validator:
Screenshot 2025-11-19 at 6 55 27 am

Parallel stream improvement for engine_getBlobsV2 versus #9426:
Screenshot 2025-11-19 at 5 00 30 pm

@siladu siladu force-pushed the optimise-getPayload-rpcs-toFastHex branch from c661321 to 1204c0b Compare November 18, 2025 14:40
In prep for using optimised HexUtil.toFastHex

Add missing EngineGetPayloadV5 unit

Signed-off-by: Simon Dudley <[email protected]>
@siladu siladu force-pushed the optimise-getPayload-rpcs-toFastHex branch from 1204c0b to ade8488 Compare November 18, 2025 14:45
engine_getBlobsV2+ and engine_getPayloadV5+

Signed-off-by: Simon Dudley <[email protected]>
@siladu siladu marked this pull request as ready for review November 19, 2025 17:01
Copilot AI review requested due to automatic review settings November 19, 2025 17:01
@siladu siladu changed the title Optimise toFastHex for engine_getPayload* Optimise engine_getPayload* and engine_getBlobsV2 Nov 19, 2025
Signed-off-by: Simon Dudley <[email protected]>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

@Gabriel-Trintinalia Gabriel-Trintinalia left a comment

Choose a reason for hiding this comment

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

LGTM

@siladu siladu enabled auto-merge (squash) November 24, 2025 11:09
@siladu siladu disabled auto-merge November 24, 2025 11:09
@siladu siladu enabled auto-merge (squash) November 24, 2025 11:10
@siladu siladu merged commit 815345b into hyperledger:main Nov 24, 2025
46 checks passed
@siladu siladu deleted the optimise-getPayload-rpcs-toFastHex branch November 24, 2025 11:48
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.

2 participants