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

improve: Reduce transaction submission retries #2128

Merged
merged 2 commits into from
Feb 26, 2025
Merged

Conversation

pxrl
Copy link
Contributor

@pxrl pxrl commented Feb 26, 2025

Because there are multiple providers for each chain, reduce the number of retries on transaction submission to speed-up the case where a transaction is actually unable to be submitted.

There are two layers of transaction submission at play - one at the quorum provider level, and one above. The quorum provider ensures that transactions are submitted successively to each provider until success or failure. For most chains, this means that transactions are submitted to 2 - 3 providers per retry. The upper layer then implements its own retry mechanism, currently at 2 retries. In cases where the transaction is ultimately unable to be submitted (i.e. underpriced replacement), this results in multiple futile submission attempts.

There are two layers of transaction submission at play - one at the
quorum provider level, and one above. The quorum provider ensures that
transactions are submitted successively to each provider until success
or failure. The upper layer then implements its own retry mechanism,
currently at 2 retries. This ultimately results in a _lot_ of attempts
to submit transactions, which causes delays when the transaction is
inherently unable to be submitted (i.e. due to nonce issues).
@pxrl pxrl merged commit 3a0db47 into master Feb 26, 2025
4 checks passed
@pxrl pxrl deleted the pxrl/dropRetries branch February 26, 2025 15:31
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.

2 participants