Skip to content

fix: issues from RPC testing#759

Merged
aloknerurkar merged 2 commits intomainfrom
rpc-fixes.6
Aug 14, 2025
Merged

fix: issues from RPC testing#759
aloknerurkar merged 2 commits intomainfrom
rpc-fixes.6

Conversation

@aloknerurkar
Copy link
Copy Markdown
Collaborator

Describe your changes

  • In some cases the sender was not getting triggered properly causing the txn to be stuck in the DB. Once a new txn comes in the trigger happens, but we need to be more aggresive.
  • Added a timeout of 10m for txns. If it takes more than that, we fail the operation.

Issue ticket number and link

Fixes # (issue)

Checklist before requesting a review

  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

Copy link
Copy Markdown
Contributor

@chrmatt chrmatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good as far as I can tell.

Comment thread tools/preconf-rpc/sender/sender.go Outdated
@aloknerurkar aloknerurkar merged commit 757693d into main Aug 14, 2025
5 checks passed
@aloknerurkar aloknerurkar deleted the rpc-fixes.6 branch August 14, 2025 12:45
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.

3 participants