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 gas profiles everywhere #8260

Open
Tracked by #8261
jakmeier opened this issue Dec 21, 2022 · 0 comments
Open
Tracked by #8261

Use gas profiles everywhere #8260

jakmeier opened this issue Dec 21, 2022 · 0 comments
Labels
A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc)

Comments

@jakmeier
Copy link
Contributor

jakmeier commented Dec 21, 2022

Gas profiles are a nearcore internal concept. It is not reflected on chain and is not observable on the protocol level.

The use of gas profiles so far is just for the RPC methods, so they can provide a detailed break down of gas costs on tx-status calls. I think they are also used by tooling for smart contract developers, so they can compare gas costs across code changes.

AFAIR, profiles currently are only collected for function calls. It's definitely not collected for transaction to receipt conversion. But I am not sure about action receipt execution for actions that are not function calls.

The goal of this issue is to always collect gas profiles. This would allow to use gas profile for a bunch of new use cases:

@jakmeier jakmeier added the A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc) label Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc)
Projects
None yet
Development

No branches or pull requests

1 participant