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.

Sign up for GitHub

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

Jump to bottom

optimize process gas #1771

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

optimize process gas #1771

wants to merge 1 commit into from

Conversation

scf0220
Copy link

@scf0220 scf0220 commented Jul 11, 2023 •
edited
Loading

There is no need to obtain gas-related information from the store every time

@scf0220 scf0220 mentioned this pull request Jul 18, 2023
Open
@scf0220
optimize handle gas
be81b83 
optimize handle gas
@scf0220 scf0220 force-pushed the optimize-handle-gas branch from 2ea25fd to be81b83 Compare July 20, 2023 04:23
@scf0220 scf0220 mentioned this pull request Jul 20, 2023
Closed
@webmaster128
Copy link
Member

So the idea here is to only get the Global once instead of letting the metering middleware get it multiple times? How much improvement can this create?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
okx
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@scf0220 @webmaster128