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

Client is too eager to declare transactions as dead #106

Open
abizjak opened this issue Dec 14, 2021 · 0 comments
Open

Client is too eager to declare transactions as dead #106

abizjak opened this issue Dec 14, 2021 · 0 comments
Labels
[Prio] Low Should be fixed if time permits but can be postponed. [Type] Bug Something isn't working

Comments

@abizjak
Copy link
Contributor

abizjak commented Dec 14, 2021

Bug Description

The client currently immediately queries the transaction after submitting it to the node. This can cause sporadic failure due to data races especially if multiple nodes are behind a load balancer/proxy.

There is no need to query immediately and adding a delay of 2s to the first query would make the behaviour more predictable.

Alternatively we could have a grace counter for errors. Say 5 failed queries (transaction absent) before we declare it truly dead.

Steps to Reproduce

Put two nodes behind a load balancer that routes each request to a different node.

Expected Result

transaction send for a valid transaction leads to "transaction finalized"

Actual Result

transaction send fails with "Transaction was absent".

┆Issue is synchronized with this Jira Task by Unito

@abizjak abizjak added [Type] Bug Something isn't working [Prio] Low Should be fixed if time permits but can be postponed. labels Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Prio] Low Should be fixed if time permits but can be postponed. [Type] Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant