Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

validator should include transaction based on gas used #1000

Closed
yihuang opened this issue Mar 16, 2022 · 2 comments
Closed

validator should include transaction based on gas used #1000

yihuang opened this issue Mar 16, 2022 · 2 comments

Comments

@yihuang
Copy link
Contributor

yihuang commented Mar 16, 2022

Proposal: validator should include transactions based on gas used

track issue for the long-term solution of #989.

Current behavior: we have a temporary solution right now

Desired behavior:

PrepareProposal in abci++
We can run the tx in the PrepareProposal phase, and truncate the preliminary transaction list according to the gas used.

Use case: [Why is this important (helps with prioritizing requests)]

Requests may be closed if we're not actively planning to work on them.

@loredanacirstea
Copy link
Contributor

Related issue: evmos/evmos#311 (blocks are filled with more transactions than can be processed, due to the block gas limit), as this can still happen now.

@github-actions
Copy link

github-actions bot commented Jun 2, 2022

This issue is stale because it has been open 45 days with no activity. Remove Status: Stale label or comment or this will be closed in 7 days.

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

No branches or pull requests

2 participants