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

zkApp transactions stuck in local mempool #14550

Closed
2 tasks done
garethtdavies opened this issue Nov 13, 2023 · 5 comments
Closed
2 tasks done

zkApp transactions stuck in local mempool #14550

garethtdavies opened this issue Nov 13, 2023 · 5 comments
Labels
bug Testworld-2-0-protocol-performance-testing Should be used to raise general issues with Testworld 2.0 network triage

Comments

@garethtdavies
Copy link
Contributor

Preliminary Checks

Description

I sent 1000 zkApp transactions (spaced at a 3-second interval). Only ~500 were included. The rest are stuck in a local mempool. However, it isn't possible to resend using o1js await tx.send(); as it will not rebroadcast the transaction:

Error: Failed to send transaction {
  statusCode: 200,
  statusText: `Couldn't send zkApp command: ["Duplicate"]`
}

Stuck tx in local mempool:

CleanShot 2023-11-13 at 08 13 41@2x

The only way to resolve this would be to restart the node, or use a different node to send.

Steps to Reproduce

Broadcast many zkApp tx, and after some time they will get dropped from other mempools.

Expected Result

Transactions are included.

Actual Result

Transactions are dropped but remain in local mempool.

How frequently do you see this issue?

Frequently

What is the impact of this issue on your ability to run a node?

High

Status

Latest testworld

Additional information

No response

@deepthiskumar
Copy link
Member

@garethtdavies this is most probably the same as the issue we were experiencing due to delay in block production (#14531) Blocks filled with zkApp transactions take longer than a slot to be broadcasted and therefore rejected, leaving all transactions in the pool.

@garethtdavies
Copy link
Contributor Author

garethtdavies commented Nov 14, 2023

Thanks. Will the proposed staged ledger fix (eluded to by Brandon) be included in time to test this on Testworld?

Edit: this one #14547

@amc-ie
Copy link
Member

amc-ie commented Nov 28, 2023

Triage update: This is linked to block production slowness/performance and should be fixed with our upcoming performance improvements in rampup7.

@dkijania
Copy link
Member

dkijania commented Jan 4, 2024

@garethtdavies rampup7 with performance improvements was released. Can you retest the issue on the newest testworld-2-0 release?

@amc-ie
Copy link
Member

amc-ie commented Jan 18, 2024

This issue should be closed with the rampup8 release. Please reopen if the issue persists on the latest version. Thank you!

@amc-ie amc-ie closed this as completed Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Testworld-2-0-protocol-performance-testing Should be used to raise general issues with Testworld 2.0 network triage
Projects
Status: Reviewed
Development

No branches or pull requests

5 participants