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 section on MEV after The Merge #7333

Merged
merged 4 commits into from
Oct 4, 2022
Merged

Conversation

emmanuel-awosika
Copy link
Contributor

Describes the risks that MEV extraction poses to Ethereum under proof-of-stake and evaluates two solutions to curbing negative impacts of MEV in post-Merge Ethereum (Proposer-Builder Separation and MEV Boost).

Related Issue

#7271

Describes the risks that MEV extraction poses to Ethereum under proof-of-stake and evaluates two solutions to curbing negative impacts of MEV in post-Merge Ethereum (Proposer-Builder Separation and MEV Boost).
@gatsby-cloud
Copy link

gatsby-cloud bot commented Aug 8, 2022

✅ ethereum-org-website-dev deploy preview ready

Copy link
Contributor

@ThreeMuskets12 ThreeMuskets12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MEV Boost isn't necessarily one of two solutions to "MEV after The Merge" as much as it is one implementation (albeit, likely the only implementation that will be widely adopted) of the Ethereum Builder API. (https://github.com/ethereum/builder-specs) I think this might be confusing to readers who might confuse Flashbot's software implementation of the Builder API (named MEV-boost) with the "MEV after The Merge" solution itself, the Builder API.

@emmanuel-awosika
Copy link
Contributor Author

emmanuel-awosika commented Aug 8, 2022

MEV Boost isn't necessarily one of two solutions to "MEV after The Merge"

I described MEV Boost as "one of two solutions" because it made sense to include a solution that people could use now, as opposed to in-protocol proposer-builder separation which is far off into the future. Yes, it implements Builder API specs, but I doubt that changes anything.

I think this might be confusing to readers who might confuse Flashbot's software implementation of the Builder API (named MEV-boost) with the "MEV after The Merge" solution itself, the Builder API.

Yes, MEV Boost is but one implementation of the Builder API, but I think the chances of readers confusing the two by reading page is low. There's no mention of MEV Boost in reference to the Builder API--the closest is a line that says, "MEV Boost uses a modified version of the Engine API that allows consensus clients to request execution payloads from execution clients."

Perhaps listing the Builder API as the other "solution" to MEV, instead of MEV Boost, would have also worked, but the latter provided a ready example of how it worked in practice--so I used it instead. And as you say, MEV Boost will likely be the most used implementation of the Builder API, so having an explainer for readers on it wouldn't hurt. (Also, the Builder API specs were based on the original MEV Boost proposal, if that's saying something).

Reworked the page to emphasize the Builder API.
@emmanuel-awosika
Copy link
Contributor Author

Hey @ThreeMuskets12, I reworked the page to better emphasize the Builder API. Let me know what you think about the additions. And thanks for pointing out the issue in the earlier version.

@@ -116,6 +116,92 @@ While many searchers are still making good money from MEV, as opportunities beco

As DeFi grows and increases in popularity, MEV may soon significantly outweigh the base Ethereum block reward. With that comes a growing possibility of selfish block remining and consensus instability. Some consider this to be an existential threat to Ethereum, and disincentivizing selfish mining is an active area of research in Ethereum protocol theory. One solution currently being explored is [MEV reward smoothing](https://ethresear.ch/t/committee-driven-mev-smoothing/10408).

## MEV after The Merge {#mev-after-the-merge}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel like this wording worked when TM was still looming --now it is in the past maybe we refer to "MEV since The Merge" or "PoS-specific MEV" or similar?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does "MEV in Ethereum Proof-of-Stake (PoS)" sound good? I like "MEV since the Merge", but it may not be ideal for SEO.

Copy link
Contributor

@jmcook1186 jmcook1186 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems good to me overall, but I'm no expert on this so have asked some researchers to take a look to confirm details.

@emmanuel-awosika
Copy link
Contributor Author

seems good to me overall, but I'm no expert on this so have asked some researchers to take a look to confirm details.

Alright, then. Thanks for reviewing.

- Changed "MEV after The Merge" to "MEV in Ethereum proof-of-stake"
- Added mevboost.org to resources section
Copy link
Contributor

@minimalsm minimalsm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Struggling to get an expert review, but I'm fine with merging this as is.

@minimalsm minimalsm merged commit 41df40f into ethereum:dev Oct 4, 2022
@corwintines corwintines mentioned this pull request Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants