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

Increase limit of events per transaction #302

Open
kroggen opened this issue Oct 26, 2023 Discussed in #254 · 0 comments · Fixed by #303
Open

Increase limit of events per transaction #302

kroggen opened this issue Oct 26, 2023 Discussed in #254 · 0 comments · Fixed by #303
Labels
proposal V4 To be released on hardfork V4

Comments

@kroggen
Copy link
Member

kroggen commented Oct 26, 2023

The current limit is 50 events / txn

When the MaxCallDepth was increased to 64 this other limit was not changed

It may be interesting to make the limit of events to be somewhat like twice the MaxCallDepth, allowing for 2 events for every contract call

On this case, if approved, the new limit would be 128 events per transaction (on V4)

Is it OK? Is there any implication of this change?
@kroggen kroggen added proposal V4 To be released on hardfork V4 labels Oct 26, 2023
@kroggen kroggen linked a pull request Oct 29, 2023 that will close this issue
increase max events per txn #303
Merged
@kroggen kroggen mentioned this issue Apr 30, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
proposal V4 To be released on hardfork V4
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

increase max events per txn aergoio/aergo
1 participant
@kroggen