Skip to content

receipts gen: dedup parallel re-exec of same block#14372

Merged
AskAlexSharov merged 5 commits into
mainfrom
r_lru_dedup
Apr 1, 2025
Merged

receipts gen: dedup parallel re-exec of same block#14372
AskAlexSharov merged 5 commits into
mainfrom
r_lru_dedup

Conversation

@AskAlexSharov

@AskAlexSharov AskAlexSharov commented Apr 1, 2025

Copy link
Copy Markdown
Collaborator
  • Latency impact: no impact. But 1 block executed once (even if multiple parallel requests) - means less CPU, means less chance to over-load server (what will increase latency).
  • Throughput impact on "many parallel requests of same block" case (most of requests hit cache): benchmarks don't show throughput difference

@AskAlexSharov AskAlexSharov mentioned this pull request Apr 1, 2025
5 tasks
@AskAlexSharov AskAlexSharov changed the title [wip] receipts gen: dedup parallel re-exec of same block receipts gen: dedup parallel re-exec of same block Apr 1, 2025
@AskAlexSharov AskAlexSharov requested a review from Giulio2002 April 1, 2025 06:09
@AskAlexSharov

Copy link
Copy Markdown
Collaborator Author

will need pick to 3.0

@AskAlexSharov AskAlexSharov merged commit cf64fd6 into main Apr 1, 2025
@AskAlexSharov AskAlexSharov deleted the r_lru_dedup branch April 1, 2025 08:07
AskAlexSharov added a commit that referenced this pull request Apr 1, 2025
- Latency impact: no impact. But 1 block executed once (even if multiple
parallel requests) - means less CPU, means less chance to over-load
server (what will increase latency).
- Throughput impact on "many parallel requests of same block" case (most
of requests hit cache): benchmarks don't show throughput difference
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.

3 participants