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

Feature Gate: add default loaded_accounts_data_size_limit #1568

Open
tao-stones opened this issue May 31, 2024 · 1 comment
Open

Feature Gate: add default loaded_accounts_data_size_limit #1568

tao-stones opened this issue May 31, 2024 · 1 comment
Labels
feature-gate Pull Request adds or modifies a runtime feature gate

Comments

@tao-stones
Copy link

SIMD

none

Description

When compute_budget instruction set_loaded_accounts_data_size_limit is not used, transaction currently has implicit limit of MAX_LOADED_ACCOUNTS_DATA_SIZE_BYTES. #1355 propose an reasonable default limit to be used instead of MAX.

New default limit could fail some transactions with TransactionError::MaxLoadedAccountsDataSizeExceeded, therefore the change should be behind a feature gate

Feature ID

CVvWw7NMVCn2Yp5RfxqrcTHXnaYyh91A2bGbkk88XXMM

Activation Method

Single Core Contributor

Deployment Considerations

Should communicate to dev community to adapt set_loaded_accounts_data_size_limit instruction. Thoughts here.

Minimum Beta Version

2.0

Minimum Stable Version

No response

Testnet Activation Epoch

No response

Devnet Activation Epoch

No response

Mainnet-Beta Activation Epoch

No response

@tao-stones tao-stones added the feature-gate Pull Request adds or modifies a runtime feature gate label May 31, 2024
@jstarry
Copy link

jstarry commented Jun 1, 2024

Is there a SIMD for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-gate Pull Request adds or modifies a runtime feature gate
Projects
None yet
Development

No branches or pull requests

2 participants