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

Increase default gas limit from 30M to 36M #30897

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

0xalpharush
Copy link

Copy link
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

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

It's a no from my side. See https://ethresear.ch/t/on-increasing-the-block-gas-limit-technical-considerations-path-forward/21225/2?u=holiman for a pretty good answer why

Leaving the PR open to hear other voices

@tynes
Copy link
Contributor

tynes commented Dec 11, 2024

It's a no from my side. See https://ethresear.ch/t/on-increasing-the-block-gas-limit-technical-considerations-path-forward/21225/2?u=holiman for a pretty good answer why

Leaving the PR open to hear other voices

If we don't speak in terms of objective measures, we will all speak past each other. It is true that the storage space required grows indefinitely and this is a ticking time bomb if nothing is done. I don't understand what the point of no return is with regards to database size. Do we have any way to gather metrics on the disk sizes of validators or would that be considered spyware? We basically need to have a projected trajectory of storage space growth in the worst case (all gas in every block is used to increase the db size the greatest amount possible, which may not be the same between implementations) and map that to when it will hit the point of no return under different gas limits. @sslivkoff has done a good job looking at the data here. My understanding is that EIP-4444 is required before we get to the point of no return, so increasing the gas limit now increases the priority of EIP-4444 being shipped to production

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants