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

send-transactions: optimize retry pool #35081

Conversation

fanatid
Copy link
Contributor

@fanatid fanatid commented Feb 5, 2024

Problem

Transactions not included in the retry pool on full utilization

Summary of Changes

  • do not insert transactions with zero max_retries to the retry pool
  • remove transactions reached max_retries in the same iteration of the loop
  • dynamically select sleep time between iterations based on last_sent_time in TransactionInfo

@mergify mergify bot added community Community contribution need:merge-assist labels Feb 5, 2024
@mergify mergify bot requested a review from a team February 5, 2024 16:09
@fanatid fanatid force-pushed the send-transactions-opt-retry-master-based branch from 71cee5b to 9ce4b11 Compare February 5, 2024 20:29
@fanatid fanatid force-pushed the send-transactions-opt-retry-master-based branch from 7d21f0e to 8422a4f Compare February 21, 2024 22:29
@willhickey
Copy link
Contributor

This repository is no longer in use. Please re-open this pull request in the agave repo: https://github.com/anza-xyz/agave

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contribution need:merge-assist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants