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

Reduce internal strucutre size in memory #260

Closed
ikopylov opened this issue Apr 29, 2023 · 0 comments · Fixed by #281
Closed

Reduce internal strucutre size in memory #260

ikopylov opened this issue Apr 29, 2023 · 0 comments · Fixed by #281
Assignees
Labels
enhancement New feature or request performance

Comments

@ikopylov
Copy link
Member

There are several places where we can reduce the size of internal structs. Since there are expected to be about 100k instances of Storage in production, even a small reduction can give good improvements in memory consumption.

  1. Bloom filter config sharing. It is expected, that all instances will have the same config. This shared value can be stored in the static field with lazy initialization (https://github.com/qoollo/pearl/blob/master/src/filter/bloom.rs#L16)
  2. FileName consumes too much memory (https://github.com/qoollo/pearl/blob/master/src/blob/core.rs#L466)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request performance
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant