You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As the FVM matures, some settings are no longer needed. Continuing support for those settings increases the complexity of the code and the amount of code paths that need to be tested.
Proposed Solution
TransactionFeesEnabled should be removed. If a situation requires the transaction fees to be disabled, they should be set to 0 instead.
MaxNumOfTxRetries is not used and should be removed.
restrictedAccountCreationEnabled will not be used anymore.
RestrictContractDeployment and RestrictContractRemoval and MemoryLimit are FVM variables and constants in the state, where the state constants take precedence. They should move to the state completely and the FVM settings removed. This can be done after canary, testnet and mainnet have the settings set in the state.
LimitAccountStorage should be always enabled, and the setting removed. (could cause problems in tests)
AllowContextOverrideByExecutionState was a temp workaround to allow the system transaction to run with more resources. An alternate solution should be found.
Some of these also require core contract changes (cleanups).
Definition of Done
Remove the mentioned settings
Check for any other FVM settings that can be removed
The text was updated successfully, but these errors were encountered:
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Problem Definition
As the FVM matures, some settings are no longer needed. Continuing support for those settings increases the complexity of the code and the amount of code paths that need to be tested.
Proposed Solution
Some of these also require core contract changes (cleanups).
Definition of Done
The text was updated successfully, but these errors were encountered: