-
Notifications
You must be signed in to change notification settings - Fork 128
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
Update limits #996
Update limits #996
Conversation
Preview is available here: |
Core PR for the new limits, stellar/stellar-core#4491 |
Preview is available here: |
1 similar comment
Preview is available here: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marked some settings as KiB. The state archival one stays at KB. In future, how about splitting them into transaction level limits and ledger wide limits?
@anupsdf I think that's probably a good idea. We can split them up in a separate PR. |
Preview is available here: |
We probably should merge this PR on October 9th when the new limits go live. |
Sounds good. @ElliotFriend are you okay to merge this on Oct. 9th? I'm out of town next week so hoping to lean on you! |
No description provided.