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

Use nodes local min gas price for prepare proposal #3016

Closed
4 tasks
cmwaters opened this issue Jan 16, 2024 · 5 comments
Closed
4 tasks

Use nodes local min gas price for prepare proposal #3016

cmwaters opened this issue Jan 16, 2024 · 5 comments
Assignees

Comments

@cmwaters
Copy link
Contributor

cmwaters commented Jan 16, 2024

Summary

The global min gas price is used as the minimum price for a transaction to enter a nodes mempool. The node still has a locally configured min price. This should be used when preparing the proposal such that all transaction included must be greater than that price

Proposal

In PrepareProposal, while looping through all transactions, if one has a gas price lower than the nodes own threshold skip over it and do not include it in the list of transactions to be proposed

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@evan-forbes
Copy link
Member

The global min gas price is used as the minimum price for a transaction to enter a nodes mempool. The node still has a locally configured min price. This should be used when preparing the proposal such that all transaction included must be greater than that price

do transactions make it in the mempool that are less than the locally configured min fee? if not, then would the filter in prepare proposal every filter anything out?

@cmwaters
Copy link
Contributor Author

do transactions make it in the mempool that are less than the locally configured min fee?

Yes so long as they are greater than the global min gas price.

The locally configured min fee just means that this proposer won't include them but they may be included by another proposer. If the transaction has too low of a fee, it will eventually be kicked.

@rootulp
Copy link
Collaborator

rootulp commented Jan 24, 2024

Sorry for the silly question but why do we want to support this?

@ninabarbakadze
Copy link
Member

update: we stopped working on this until we figure out next steps including whether or not we even want this feature.

@rootulp
Copy link
Collaborator

rootulp commented Feb 22, 2024

I think we can close this as won't do b/c we decided to not modify the local min gas price. Copying from Slack:

Decision to not modify the purpose of the local config. It can still be used to increase the min fee to be accepted in the mempool.

Please re-open if I misinterpreted @ninabarbakadze @cmwaters

@rootulp rootulp closed this as not planned Won't fix, can't repro, duplicate, stale Feb 22, 2024
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

No branches or pull requests

4 participants