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

Problem: transaction ordering #2263

Open
VictorTrustyDev opened this issue Jan 13, 2024 · 5 comments
Open

Problem: transaction ordering #2263

VictorTrustyDev opened this issue Jan 13, 2024 · 5 comments
Labels
feature request Add functionality to the code

Comments

@VictorTrustyDev
Copy link
Contributor

VictorTrustyDev commented Jan 13, 2024

Context

Current issue:

  • When an account broadcast many transactions at a time (with nonce increament), there is high chance that most of the txs will be failed to execute due to nonce check, that caused by un-ordered by sequence when executing txs.
  • When trace transaction, the data is not accurate if there is Cosmos Txs before or in the middle of the txs that modified store and trace tx can not simulate that.

That caused by the reason that current Tendermint/CometBFT version we are using, does not support re-order the transactions.

Possible solution:

  • ABCI 2.0. As I understood, PrepareProposal would help us with the ability to modify the txs bytes, that we can used to re-order the txs as the following orders:
    • EVM txs first
    • Within EVM txs, follow Ethereum's order.
    • Any others if applicable

Problem may occurs with the above solution, need to aware:

  • Needed core Cosmos txs might not be able to be executed like IBC messages,...

Expected

Copy link

linear bot commented Jan 23, 2024

ENG-2466 Issue 2263

@hanchon hanchon added the feature request Add functionality to the code label Jan 24, 2024
@hanchon
Copy link
Contributor

hanchon commented Jan 24, 2024

With ABCI2.0 we can fix it 🚀

Copy link

This issue is stale because it has been open 45 days with no activity. Remove Status: Stale label or comment or this will be closed in 7 days.

@MalteHerrmann
Copy link
Contributor

SDK v0.50 and ABCI++ are on the way

@mikebraver
Copy link

Did the latest version available of SDK and ABCI already solved this problem?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Add functionality to the code
Projects
None yet
Development

No branches or pull requests

4 participants