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

Additional memory and storage savings are possible by deduplicating Cadence dictionary type info #358

Closed

fxamacker opened this issue Nov 22, 2023 · 1 comment

Assignees

Labels

enhancement performance

Member

fxamacker commented Nov 22, 2023 •

edited

Loading

Updates #292, onflow/cadence#1854, onflow/flow-go#1744

I think we can deduplicate Cadence dictionary type info, which would further reduce total payload size.

PR #342 added atree inlining which makes data deduplication feasible. Same PR also added data deduplication for:

Cadence array type info
Cadence composite type info

After examining real data from mainnet, I think the savings can be worthwhile by deduplicating Cadence dictionary type info.

fxamacker added enhancement performance labels

fxamacker self-assigned this

fxamacker mentioned this issue

Reduce RAM and persistent storage by deduplicating inlined dict type info #369

Merged

6 tasks

Member Author

fxamacker commented Mar 14, 2024

Closed by #369

fxamacker closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment