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

ExecutionCancelledDueToSharedObjectCongestion #19887

Open
wowok-ai opened this issue Oct 17, 2024 · 4 comments
Open

ExecutionCancelledDueToSharedObjectCongestion #19887

wowok-ai opened this issue Oct 17, 2024 · 4 comments
Assignees
Labels
doc-issue Issue submitted using the Doc issue template

Comments

@wowok-ai
Copy link

https://testnet.suivision.xyz/txblock/3gwkhNH8deHPVBsagv7fvssC1QgtUH2AGXUT1MXgMEAS

This error did not occur when executing this transaction before. What is happening?

@wowok-ai wowok-ai added the doc-issue Issue submitted using the Doc issue template label Oct 17, 2024
@mwtian
Copy link
Member

mwtian commented Oct 17, 2024

I think the recommendation would be to use a more accurate gas budget (0.0021 instead of 0.0217264). Right now the system is unnecessarily treating this transaction as an expensive transaction that cannot be executed.

@mystenmark
Copy link
Contributor

Hi - unfortunately we had to adjust our congestion control mechanism due to the current FOMO mining which is using large PTBs to stuff lots of work into a single transaction.

Any transaction that either has a budget of less than 0.018 SUI (18.5M MIST), or that has few enough MoveCalls / inputs will be able to run. "Few enough" means: sum of MoveCall commands + MoveCall Input object arguments < 46

Based on analyzing past checkpoints we did not find very many transactions that would be impacted by this, although we knew there would be a few, and we apologize.

The good news is that this is a temporary mitigation - in the next release we will add support for larger / higher budget transactions to run unimpeded once again.

@Programmierus
Copy link

You guys know, right, that under "not very many" fall indeed quite a few transactions that exchange aggregators produce with longer routes?

Impact example can be seen here.

@mwtian
Copy link
Member

mwtian commented Oct 17, 2024

@Programmierus yes we are aware of the issue and sorry for the problems this has caused. An new protocol upgrade targeting tomorrow is in the works to raise the limit to avoid affecting these transactions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-issue Issue submitted using the Doc issue template
Projects
None yet
Development

No branches or pull requests

8 participants