Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(op-node): add pre fetch receipts logic #57

Merged
merged 12 commits into from
Oct 13, 2023

Conversation

welkin22
Copy link
Contributor

@welkin22 welkin22 commented Sep 21, 2023

[Description]
The process of fetching receipts is notably time-consuming. This issue can escalate further when the L1 network is busy, leading to an even more protracted duration of this logic. To mitigate these time constraints, it is imperative for the sequencer to execute this logic preemptively and store it in a cache. This will help prevent any negative impact on the block generation process.

[Rationale]
The process of fetching receipts is notably time-consuming.

[Example]
There are no changes to cli or api, only op-node internal logic changes.

[Changes]

  • Added the GoOrUpdatePreFetchReceipts method to enable asynchronous goroutine for obtaining receipts in advance
  • Call the GoOrUpdatePreFetchReceipts logic in the EngineQueue.Reset method
  • Some interface and test adaptation work

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2023

This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Oct 6, 2023
@owen-reorg owen-reorg removed the Stale label Oct 7, 2023
op-node/testutils/mock_l1.go Show resolved Hide resolved
@welkin22 welkin22 merged commit 628b517 into bnb-chain:develop Oct 13, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants