Skip to content
This repository was archived by the owner on Oct 25, 2024. It is now read-only.

Conversation

@dvush
Copy link
Contributor

@dvush dvush commented Dec 23, 2022

📝 Summary

Using types.Sender(signer, tx) instead of signer.Sender(tx) is much better because the former caches the result while the latter does not. It takes a significant amount of time to do it all the time. (around 10% of the block building time is wasted because of the changed line in applyTransactionWithBlacklist).

The line in commitPayoutTx does not matter because we see that tx only once but its still nice to have one way to do it in the code.

@Ruteri Ruteri merged commit f5c766f into flashbots:main Dec 23, 2022
avalonche pushed a commit that referenced this pull request Dec 23, 2022
avalonche pushed a commit that referenced this pull request Feb 7, 2023
* sim bundles in parallel
* add cache for failed bundles too
avalonche pushed a commit that referenced this pull request Feb 7, 2023
avalonche pushed a commit that referenced this pull request Mar 9, 2023
* sim bundles in parallel
* add cache for failed bundles too
avalonche pushed a commit that referenced this pull request Mar 9, 2023
avalonche pushed a commit that referenced this pull request Mar 15, 2023
* sim bundles in parallel
* add cache for failed bundles too
avalonche pushed a commit that referenced this pull request Mar 15, 2023
avalonche pushed a commit that referenced this pull request Mar 17, 2023
* sim bundles in parallel
* add cache for failed bundles too
avalonche pushed a commit that referenced this pull request Mar 17, 2023
avalonche pushed a commit that referenced this pull request Mar 22, 2023
* sim bundles in parallel
* add cache for failed bundles too
avalonche pushed a commit that referenced this pull request Mar 22, 2023
avalonche pushed a commit that referenced this pull request Jul 6, 2023
* sim bundles in parallel
* add cache for failed bundles too
avalonche pushed a commit that referenced this pull request Jul 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants