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

Limit keys and values in storage, limit state part size #1805

Closed
1 task
mikhailOK opened this issue Dec 2, 2019 · 13 comments
Closed
1 task

Limit keys and values in storage, limit state part size #1805

mikhailOK opened this issue Dec 2, 2019 · 13 comments
Labels
A-storage Area: storage and databases P-low Priority: low T-core Team: issues relevant to the core team

Comments

@mikhailOK
Copy link
Contributor

mikhailOK commented Dec 2, 2019

@ilblackdragon ilblackdragon added this to the MainNet milestone Dec 8, 2019
@MaksymZavershynskyi MaksymZavershynskyi added A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc) runtime-stability labels Dec 9, 2019
@MaksymZavershynskyi
Copy link
Contributor

@evgenykuzyakov The first two items are implemented by #1913 correct?

@MaksymZavershynskyi MaksymZavershynskyi added P-high Priority: High P-critical Priority: critical and removed runtime-stability P-high Priority: High labels Jan 20, 2020
@MaksymZavershynskyi MaksymZavershynskyi added A-storage Area: storage and databases P-high Priority: High and removed A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc) P-critical Priority: critical labels Feb 4, 2020
@MaksymZavershynskyi
Copy link
Contributor

MaksymZavershynskyi commented Feb 4, 2020

@Kouprin the PR does not seem to be blocked anymore. Assigning it to you, so that you can merge and close it. Also lowered the priority to P1 since the initial assignment of P0 was incorrect, I think.

@Kouprin
Copy link
Member

Kouprin commented Feb 4, 2020

@nearmax what is happened that you assigned this task on me?

@MaksymZavershynskyi
Copy link
Contributor

This issue is waiting for #1771 to be done, since you are the person working on it I assigned it to you. Once the PR is closed this issue should be closed too.

@Kouprin
Copy link
Member

Kouprin commented Feb 4, 2020

@nearmax nope, #1771 is blocked for merging because of this issue.

@Kouprin Kouprin removed their assignment Feb 4, 2020
@MaksymZavershynskyi
Copy link
Contributor

@nearmax nope, #1771 is blocked for merging because of this issue.

Got it.

@MaksymZavershynskyi
Copy link
Contributor

This is not needed before the Mainnet, since we will be able to introduce this variable into config later, together with other new variables. Moving to Ice Box

@stale
Copy link

stale bot commented Jul 1, 2021

This issue has been automatically marked as stale because it has not had recent activity in the last 2 months.
It will be closed in 7 days if no further activity occurs.
Thank you for your contributions.

@stale stale bot added the S-stale label Jul 1, 2021
@bowenwang1996
Copy link
Collaborator

I believe we have all the limiters we need. Just need to investigate the upper bound for state parts.

@stale stale bot removed the S-stale label Jul 1, 2021
@bowenwang1996 bowenwang1996 added T-core Team: issues relevant to the core team and removed P-high Priority: High labels Jul 1, 2021
@stale
Copy link

stale bot commented Sep 29, 2021

This issue has been automatically marked as stale because it has not had recent activity in the last 2 months.
It will be closed in 7 days if no further activity occurs.
Thank you for your contributions.

@stale stale bot added the S-stale label Sep 29, 2021
@stale
Copy link

stale bot commented Dec 29, 2021

This issue has been automatically marked as stale because it has not had recent activity in the last 2 months.
It will be closed in 7 days if no further activity occurs.
Thank you for your contributions.

@stale
Copy link

stale bot commented Mar 31, 2022

This issue has been automatically marked as stale because it has not had recent activity in the last 2 months.
It will be closed in 7 days if no further activity occurs.
Thank you for your contributions.

@stale stale bot added the S-stale label Mar 31, 2022
@Longarithm
Copy link
Member

Boundary for state part size is tracked in #1786.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-storage Area: storage and databases P-low Priority: low T-core Team: issues relevant to the core team
Projects
None yet
Development

No branches or pull requests

6 participants