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

Remove legacy EVM limit #2647

Merged
merged 1 commit into from
Oct 31, 2023
Merged

Remove legacy EVM limit #2647

merged 1 commit into from
Oct 31, 2023

Conversation

Bushstar
Copy link
Member

@Bushstar Bushstar commented Oct 31, 2023

Summary

  • Remove legacy EVM TX limit

Notes

  • Testnet and changi are currently running with rc that have this legacy limit.

  • The options are:

    1. Fork guard (this is not helpful for anyone, as higher limits can't be tested on testnet which defeats the purpose of the removal).
    2. Launch mainnet with 32k limit, until next fork which is disruptive for projects that have higher requirements.
    3. Find a middle path.
  • A possible mid-path: In order to avoid disruption:

    • Set MAX_OP_RETURN_EVM_ACCEPT to < 32k (~28k).
    • This is the network config that superseded the legacy hard limit. This ensures rc1 and rc2 will continue to follow the network until a later point.
    • Pull MAX_OP_RETURN_EVM_ACCEPT to higher values at a later point (likely after mainnet is live to ensure everyone is on stable)

Additional Info

  • The new limit defaults are
    static const uint64_t MAX_OP_RETURN_CORE_ACCEPT = 1024;
    static const uint64_t MAX_OP_RETURN_DVM_ACCEPT = 4096;
    static const uint64_t MAX_OP_RETURN_EVM_ACCEPT = 65536;
    .
  • Both the legacy and new limits are derived from analyzing the Ethereum mainnet.
  • Legacy hard limit of 32k was designed to be > 99% percentile on ETH mainnet.
  • New configurable limit of 64k was designed to be > 99.9x% percentile on ETH mainnet.

Implications

  • Storage

    • Database reindex required
    • Database reindex optional
    • Database reindex not required
    • None
  • Consensus

    • Network upgrade required
    • Includes backward compatible changes
    • Includes consensus workarounds
    • Includes consensus refactors
    • None

@prasannavl prasannavl merged commit 004f918 into master Oct 31, 2023
@prasannavl prasannavl deleted the bush/remove-legacy-limit branch October 31, 2023 12:34
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.

3 participants