Skip to content

Feature Gate: remove fee-rate-governor from fee calculation path #5782

Open
@tao-stones

Description

@tao-stones

SIMD

n/a

Description

Historically FeeRateGovernor is created and advanced from parent for every new bank. Its original intention is to adjust lamports_per_signature based on cluster throughput, which is no longer the case, lamports_per_signature has been statically 5,000 since 1.8 (or even earlier). However FeeRateGovernor still has an active side effect: its lamports_per_signature is used by hundreds of tests to set zero fee for test transactions. Tho same can be achieved without having to store bank.fee_rate_governor.lamports_per_signature to blockhash_queue. Doing this pave the way to eventually sunset FeeRateGovernor from bank, and its snapshots.

However change what's stored into RecentBlockHashs account changes account-delta-hash, which is part of BankHash (until LTdLt9Ycbyoipz5fLysCi1NnDnASsZfmJLJXts5ZxZz). Therefore this change requires feature gate (or wait till LTdLt9Ycbyoipz5fLysCi1NnDnASsZfmJLJXts5ZxZz is activated everywhere).

Feature ID

2dsTvEsgS6K5eV82Bn6hNCDm76rZUWcWeKvHjGqtUmgE

Activation Method

Single Core Contributor

Deployment Considerations

Its a internal refactoring without functional change for users

Minimum Beta Version

2.2.8

Minimum Stable Version

No response

Testnet Activation Epoch

No response

Devnet Activation Epoch

No response

Mainnet-Beta Activation Epoch

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    feature-gatePull Request adds or modifies a runtime feature gate

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions