Max limit on chunk size

[MaksymZavershynskyi]

As suggested by @mikhailOK , we need to introduce two types of limits into our protocol for chunks:

• Limit the total size of the transactions in bytes;
• Limit the total state diff in bytes (how much the state is changed by an individual chunk);

Both of these limits will allow to early filter out state sync and chunk messages on a network level. Currently, this is partially limited by the gas limit of the chunk, however there are two issues with it:

• it is complex and laborious to maintain formulas that compute from the gas limit the size of the transactions in bytes, and the size of the state diff in bytes;
• 10^15 gas might allow the total transactions sizes that are too large and we want to avoid.

CC @SkidanovAlex , @bowenwang1996 , @evgenykuzyakov

[SkidanovAlex]

Why is the total diff of the state change matter?

[MaksymZavershynskyi]

Please disregard. I thought it was so that OOM cannot be caused by we can have at most ~32MiB of state diff per chunk, which is 3.2GiB for 100 shards which should be tolerable.

[bowenwang1996]

Depending on how we want to implement this, #3274 may need to be addressed first.

[stale]

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]

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]

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.