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

add call sequence mutation that removes all reverting txs in the sequences #1264

Open
0xalpharush opened this issue May 25, 2024 · 1 comment · May be fixed by #1312
Open

add call sequence mutation that removes all reverting txs in the sequences #1264

0xalpharush opened this issue May 25, 2024 · 1 comment · May be fixed by #1312

Comments

@0xalpharush
Copy link

0xalpharush commented May 25, 2024

Describe the desired feature

For long call sequences, re-executing reverted transactions probably adds more overhead than it is worth, considering it is unlikely that tx level mutations will make the majority of them successful e.g. 99% of a call sequence with a length of 500 is failing. It would make more sense to shrink the call sequence some portion of the time to only include successful transaction and insert/extend

@ggrieco-tob
Copy link
Member

I think this should be good, even if executed once in a while, to avoid all the sequence to "saturate".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants