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
Does AddMevBundle need to use global pool mutex ? #7
Comments
I think it's a good idea! |
awesome ty for heads up! will try and get a PR out this coming week |
Closed
avalonche
pushed a commit
that referenced
this issue
Feb 7, 2023
avalonche
pushed a commit
that referenced
this issue
Mar 9, 2023
avalonche
pushed a commit
that referenced
this issue
Mar 15, 2023
avalonche
pushed a commit
that referenced
this issue
Mar 17, 2023
avalonche
pushed a commit
that referenced
this issue
Mar 22, 2023
avalonche
pushed a commit
that referenced
this issue
Jul 6, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
AddMevBundle
surely needs to lock for atomicity of adding to the bundle pool (pool.mevBundles
), but as far as I can tell it doesn't interact with the rest of the txpool so should be okay to use something likepool.bundleMu.Lock()
? Happy to implement if so 😃The text was updated successfully, but these errors were encountered: