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

Propose changes FRP15 #57

Merged
merged 3 commits into from
Jun 16, 2021
Merged

Conversation

casparschwa
Copy link
Contributor

@casparschwa casparschwa commented Jun 14, 2021

As discussed, @barnabemonnot and myself went through the FRP. Proposing the changes with this PR.

@obadiaa obadiaa requested a review from fiiiu June 14, 2021 11:33
@obadiaa
Copy link
Contributor

obadiaa commented Jun 14, 2021

Your edits look great, very excited about this! Thank you @casparschwa and @barnabemonnot 🔥

2 points/questions:

  1. It'd be great to get a sense of timeline / any deadlines you think should be observed?

  2. A point I mentioned on Discord that I'll reiterate here:

this ethereum/consensus-specs#2454 also worries me.
Especially what Micah mentions around the difficulty of socialization of tx_fees/MEV rewards for decentralized staking pools:
This is because any attempts at socialization of the block reward by anyone other than the block producer will result in side channel payment systems springing up, which is bad for the whole ecosystem. If this holds true then the early results of exacerbated inequality from the uneven distribution of MEV rewards might not be as easily smoothed out by pooling.

@fiiiu
Copy link
Member

fiiiu commented Jun 16, 2021

Looks great, thanks!

@fiiiu fiiiu merged commit b5cc501 into flashbots:main Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants