-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
FeeHistory estimator #13833
FeeHistory estimator #13833
Conversation
40c3ad8
to
caaaf2d
Compare
@huangzhen1997 Let me explain why this is not a very realistic scenario: |
Quality Gate passedIssues Measures |
* Introduce universal estimator * Fixes * Use WeiMin to cap bump price * Update connectivity logic * Fix error * Fixes * Cover an edge case when enforcing limits * Add changeset * Update mempool check logic * Update config names * Convert Universal Estimator to service * Client changes to support UE * Introduce configs * Update mocks * Fix lint * Fix test cases * Fix mockery * Fix test cases * Update comment * Fix Start/Close sync issue * Address feedback * Fix lint * Fix lint * More changes * Add more comments * Fix merge conflicts * Update CONFIG * Rename to FeeHistory estimator * Rename * Exclude zero priced priority fees * Remove HasMempool * Remove testing commit * Fixes * Add DefaultJitter * Add optimizations * Fix testscripts * Fix name * Update error messages
Fixes: https://smartcontract-it.atlassian.net/browse/BCI-3747
Introduces a new gas estimator that consolidates the functionalities of
Suggested Price
andBlock History
estimators.For legacy transactions it utilized
eth_gasPrice
and for dynamic transactionseth_feeHistory
.