Caplin: add get blobs support (fusaka)#17829
Merged
Merged
Conversation
Contributor
Author
|
@copilot lemme ping ya |
Contributor
|
@Giulio2002 I've opened a new pull request, #17830, to work on those changes. Once the pull request is ready, I'll request review from you. |
Contributor
Author
|
ops wrong place |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for retrieving blob data from the execution layer's transaction pool to optimize blob data availability checks. The main goal is to avoid redundant blob retrieval from the P2P network when blobs are already available in the txpool.
Key changes:
- Added a new
GetBlobsmethod to theExecutionEngineinterface for querying blobs from the txpool - Modified fork choice logic to check txpool for blobs before falling back to P2P retrieval
- Updated
ExecutionClientDirectconstructor to accept a txpool client parameter
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| node/eth/backend.go | Passes txpool client to ExecutionClientDirect constructor |
| cl/phase1/execution_client/interface.go | Adds GetBlobs method to ExecutionEngine interface |
| cl/phase1/execution_client/execution_client_direct.go | Implements GetBlobs to query txpool for blob data |
| cl/phase1/execution_client/execution_client_rpc.go | Adds stub GetBlobs implementation returning nil |
| cl/phase1/execution_client/execution_engine_mock.go | Auto-regenerated mock with new GetBlobs method |
| cl/phase1/forkchoice/on_block.go | Adds logic to check EL txpool for blobs before P2P retrieval |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
domiwei
approved these changes
Nov 10, 2025
mh0lt
pushed a commit
that referenced
this pull request
Nov 10, 2025
just add getBlobs support really, nothing much, nothing less.
yperbasis
added a commit
that referenced
this pull request
Nov 11, 2025
Cherry pick #17829 to `release/3.2` Co-authored-by: Giulio rebuffo <giulio.rebuffo@gmail.com>
marcello33
added a commit
to 0xPolygon/erigon
that referenced
this pull request
Feb 18, 2026
* [r32] Caplin: add get blobs support (fusaka) (erigontech#17840) Cherry pick erigontech#17829 to `release/3.2` Co-authored-by: Giulio rebuffo <giulio.rebuffo@gmail.com> * [3.2] prune: 500ms pruning timeout for stage_exec on ChainTiptar (erigontech#17890) to increase prune timeout 2x as a hotfix for reported prune issue on ethmainnet will create bigger epic today * [3.2] remove `amoy` chain from `sync-from-scratch-test` list (erigontech#17921) * cp Integrity check: compare state progress and block progress (erigontech#18002) issue: erigontech#17911 * [3.2] version (erigontech#18057) * [r3.2] rpc: fix txpool_content crash: unknown (erigontech#18119) Cherry-pick erigontech#18111 * [3.2] Bump max blob size (erigontech#18329) Previous limit was not enough for a 6 blob EIP-4844 transaction with commitments and proofs. The theoretical max size is ~824 KB. * Revert "[3.2] remove `amoy` chain from `sync-from-scratch-test` list (erigontech#17921)" This reverts commit 61d4ffc. * lisovo: CLZ opcode and P256 gas cost changes (#117) * tbdhf: CLZ opcode and P256 gas cost changes * chore: rename to lisovo * feat: flexible base fee on lisovo HF (#118) * chore: add amoy hf block number * fix: lint --------- Co-authored-by: Lucca Martins <lucca_martins30@yahoo.com.br> * chore: bump version * core, execution, polygon: lisovoPro HF and version bump * core: lisovo-lisovoPro instructionset test * core, execution, polygon: lisovoPro HF and version bump (#123) * core, execution, polygon: lisovoPro HF and version bump * core: lisovo-lisovoPro instructionset test * lisovo HF and version bump --------- Co-authored-by: Andrew Ashikhmin <34320705+yperbasis@users.noreply.github.com> Co-authored-by: Giulio rebuffo <giulio.rebuffo@gmail.com> Co-authored-by: Alex Sharov <AskAlexSharov@gmail.com> Co-authored-by: sudeepdino008 <sudeepdino008@gmail.com> Co-authored-by: canepat <16927169+canepat@users.noreply.github.com> Co-authored-by: Shoham Chakraborty <shhmchk@gmail.com> Co-authored-by: Pratik Patil <pratikspatil024@gmail.com> Co-authored-by: Krishang <109511742+kamuikatsurgi@users.noreply.github.com> Co-authored-by: Lucca Martins <lucca_martins30@yahoo.com.br>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
just add getBlobs support really, nothing much, nothing less.