Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Transactions are delayed in the network when a fork occurs #4107

Closed
lsaether opened this issue Nov 14, 2019 · 2 comments · Fixed by #4250
Closed

Transactions are delayed in the network when a fork occurs #4107

lsaether opened this issue Nov 14, 2019 · 2 comments · Fixed by #4250
Labels
I3-bug The node fails to follow expected behavior. U1-asap No need to stop dead in your tracks, however issue should be addressed as soon as possible.
Milestone

Comments

@lsaether
Copy link
Contributor

While injecting transactions in recent versions of a Substrate-based chain I've noticed transactions get delayed in the tx-pool when a fork occurs. The strange thing about this delay is that it is actually pretty long -- 238 blocks on the last time occurred. By saying "delay" I mean that the transactions are no longer on the node I initially injected them to, but form what I can tell are floating around the network and not being included for a while.

Unfortunately I don't have any logs from when this happened and it may be hard to reproduce. If someone has any ideas for trying how I can best provide more information I would be happy to know.

@bkchr
Copy link
Member

bkchr commented Nov 14, 2019

@tomusdrw any ideas?

@tomusdrw
Copy link
Contributor

What version was it? Adding transactions from retracted blocks back to the pool was added some time already #3562.
I suspect that the issue is caused by the same thing as slow transaction pool culling. Receiving notification of imported block is significantly delayed - either we are doing something very heavy and need a lock (potentially transaction pool clean up might take long) or the task get's starved.

Would be good to collect logs with -l=txpool=debug with the latest version of substrate (we already do that for our nodes) - this will allow to figure out what's going on.

@gavofyork gavofyork added this to the 2.1 milestone Nov 29, 2019
@gavofyork gavofyork added I3-bug The node fails to follow expected behavior. U1-asap No need to stop dead in your tracks, however issue should be addressed as soon as possible. labels Nov 29, 2019
@gavofyork gavofyork modified the milestones: 2.1, Next Point Release Nov 29, 2019
@gnunicorn gnunicorn modified the milestones: Next Point Release, Polkadot Mar 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
I3-bug The node fails to follow expected behavior. U1-asap No need to stop dead in your tracks, however issue should be addressed as soon as possible.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants