Skip to content

chore: remove unused return value from dispatch_with_chunking#23341

Merged
pepyakin merged 1 commit into
mainfrom
pep/cleanup-dispatch-with-chunking-return-type
Apr 2, 2026
Merged

chore: remove unused return value from dispatch_with_chunking#23341
pepyakin merged 1 commit into
mainfrom
pep/cleanup-dispatch-with-chunking-return-type

Conversation

@decofe
Copy link
Copy Markdown
Member

@decofe decofe commented Apr 2, 2026

dispatch_with_chunking returned a usize chunk count that no caller used. Changed to return () and removed the counter tracking.

Prompted by: pep

Co-authored-by: Sergei Shulepov <2205845+pepyakin@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4d55-7f04-72be-9b29-52a0aa9bcb65
@decofe decofe added the A-engine Related to the engine implementation label Apr 2, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in Reth Tracker Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

⚠️ Changelog not found.

A changelog entry is required before merging. We've generated a suggested changelog based on your changes:

Preview
---
reth-engine-tree: patch
---

Removed unused return value from `dispatch_with_chunking` in the multiproof payload processor.

Add changelog to commit this to your branch.

@pepyakin pepyakin changed the title refactor(engine): remove unused return value from dispatch_with_chunking chore: remove unused return value from dispatch_with_chunking Apr 2, 2026
@pepyakin pepyakin added this pull request to the merge queue Apr 2, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to In Progress in Reth Tracker Apr 2, 2026
Merged via the queue into main with commit abdba93 Apr 2, 2026
38 checks passed
@pepyakin pepyakin deleted the pep/cleanup-dispatch-with-chunking-return-type branch April 2, 2026 09:18
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Reth Tracker Apr 2, 2026
github-merge-queue Bot pushed a commit to tempoxyz/tempo that referenced this pull request Apr 3, 2026
Automated nightly update of reth dependencies from `paradigmxyz/reth`
main branch.

## Upstream reth changes


[`f8efc76...c82d435`](paradigmxyz/reth@f8efc76...c82d435)

🔗 Amp thread:
https://ampcode.com/threads/T-019d517e-981b-75fd-b3cf-8ad2455f38f6
**RPC**
- Remove `0x` prefix from `admin_peers` id and return keccak256 node ID
in `admin_nodeInfo.id` to match go-ethereum format
([#23318](paradigmxyz/reth#23318),
[#23319](paradigmxyz/reth#23319))
- Apply count filter only after `after` is consumed in pagination
([#23338](paradigmxyz/reth#23338))
- Pre-allocate vectors in `eth_feeHistory`
([#23334](paradigmxyz/reth#23334))
- Add `bal` RPC methods
([#23330](paradigmxyz/reth#23330))

**Engine**
- Include backpressure in reported `persistence_wait`, make wait-time
mimic CL slot time
([#23308](paradigmxyz/reth#23308))

**Perf**
- Reduce cacheline ping-pong in workers availability
([#23321](paradigmxyz/reth#23321))

**Net**
- Avoid itertools `Format` panic in tracing log
([#23331](paradigmxyz/reth#23331))

**DB**
- Cap `storage_v2` unwind history by MDBX tip
([#23335](paradigmxyz/reth#23335))

**Bench**
- Add warmup step for big blocks mode
([#23323](paradigmxyz/reth#23323))
- Make payload handling ethereum-only
([#23324](paradigmxyz/reth#23324))
- Use repository instead of GitHub Actions cache to track hourly runs
([#23306](paradigmxyz/reth#23306))
- Upload nightly regression results to ClickHouse
([#23344](paradigmxyz/reth#23344))

**Chore**
- Migrate `allow(clippy::` to `expect(clippy::`
([#23340](paradigmxyz/reth#23340))
- Remove unused return value from `dispatch_with_chunking`
([#23341](paradigmxyz/reth#23341))
- Update nixpkgs to 25.11 and refresh flake inputs
([#23342](paradigmxyz/reth#23342))

## Migrations

🔗 Amp thread:
https://ampcode.com/threads/T-019d517e-baf0-727b-a57e-0488240af973
- **Reth dependency bump**: All `reth-*` git dependencies updated from
rev `f8efc76` to `c82d435`
- **New `GetBlockAccessList` trait impl**: `TempoEthApi` now implements
`GetBlockAccessList` (imported from `reth_rpc_eth_api::helpers::bal`),
likely a new required trait added upstream in the reth update

[GitHub
Workflow](https://github.com/tempoxyz/tempo/actions/runs/23933027282)

---------

Co-authored-by: Arsenii Kulikov <klkvrr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-engine Related to the engine implementation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants