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 "MEV" page to developer docs [Resolves #3610] #3704

Merged
merged 4 commits into from
Sep 7, 2021
Merged

Add "MEV" page to developer docs [Resolves #3610] #3704

merged 4 commits into from
Sep 7, 2021

Conversation

nathanhleung
Copy link
Contributor

@nathanhleung nathanhleung commented Aug 24, 2021

Description

This PR adds a page about miner extractable value (MEV) to the developer docs.

Let me know what you think — I am also sending the deploy preview out to other folks I know in the MEV space for more feedback.

Related Issue

#3610

Link to deploy preview of the page: https://deploy-preview-3704--ethereumorg.netlify.app/en/developers/docs/miner-extractable-value/

@minimalsm
Copy link
Contributor

Thanks for the PR @nathanhleung

Overall it looks really good. Got a few subject matter experts having a look before we merge because MEV isn't something I'm knowledgeable about 🙂

@minimalsm
Copy link
Contributor

Some feedback I received:

It all seems correct to me. Maybe it would be nice to mention that the M in MEV now usually stands for maximal, not miner, and with that also that this is not a PoW problem only
Otherwise it seems like a good overview that touches on most important aspects

@nathanhleung
Copy link
Contributor Author

Thanks for the feedback! I am still waiting to hear suggestions from some of my friends in the MEV space, but I will let you know once I make the changes they suggest and this PR is ready to be merged.

Copy link
Contributor

@timbeiko timbeiko left a comment

Choose a reason for hiding this comment

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

Very good overview! Left a couple minor comments.


### The long tail {#mev-examples-long-tail}

DEX arbitrage, liquidations, and sandwich trading are all very well-known MEV opportunities and are unlikely to be profitable for new searchers. However, there is a huge long tail of less well-known MEV opportunities (NFT MEV is arguably one such opportunity).
Copy link
Contributor

Choose a reason for hiding this comment

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

s/less well-known/lesser known

@nathanhleung
Copy link
Contributor Author

Thank you for the thorough review! I've been busy this week, but just wanted to comment to say this PR is still on my mind and I hope to get to it fully this weekend.

@minimalsm
Copy link
Contributor

Thank you for the thorough review! I've been busy this week, but just wanted to comment to say this PR is still on my mind and I hope to get to it fully this weekend.

@nathanhleung no worries, thanks for keeping us in the loop 🙏

@nathanhleung
Copy link
Contributor Author

nathanhleung commented Sep 4, 2021

Made the changes y'all requested, let me know if I missed anything or you have any other suggestions. Thanks again for reviewing!

Copy link
Member

@samajammin samajammin left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks again for this 🙏.

Made a handful of small tweaks - will accept those & merge.

src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved

## MEV extraction {#mev-extraction}

While in theory, MEV accrues entirely to miners, because miners are the only party that can guarantee the execution of a profitable MEV opportunity (at least in Eth1 — this will change in Eth2), in practice, a large portion of MEV is extracted by independent network participants referred to as "searchers." Searchers run complex algorithms on blockchain data to detect profitable MEV opportunities and have bots to automatically submit those profitable transactions to the network.
Copy link
Member

Choose a reason for hiding this comment

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

at least in Eth1 — this will change in Eth2

How so? Worth expanding on? Is this change in Eth2 just validators vs. miners, or something else?

src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/mev/index.md Outdated Show resolved Hide resolved
@samajammin samajammin merged commit 735f7ea into ethereum:dev Sep 7, 2021
@minimalsm minimalsm mentioned this pull request Sep 9, 2021
@nathanhleung nathanhleung deleted the nathanhleung/mev branch September 15, 2021 03:20
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.

5 participants