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

Improve Purge in ComputedRegistry & ReplicaRegistry #33

Closed
alexyakunin opened this issue Jul 19, 2020 · 1 comment
Closed

Improve Purge in ComputedRegistry & ReplicaRegistry #33

alexyakunin opened this issue Jul 19, 2020 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@alexyakunin
Copy link
Collaborator

There is a number of problems:

  1. Default capacity should be ~ 100-200 in WASM (it's almost 8K now - I tuned it for server-side scenarios)
  2. Use precise counter instead of stochastic counter in WASM (1 core = no point)
  3. Use a separate dict + maybe something similar to generations for strong refs. Ideally, strong refs should be replaced to weak ones as quickly as possible & Purge cycles in this part must be fairly frequent (1s or so), otherwise -- combined w/ a huge capacity & stochastic counters -- it leads to issues like Investigate memory leak in Blazor apps #7 .
@alexyakunin alexyakunin added the help wanted Extra attention is needed label Jul 19, 2020
@alexyakunin
Copy link
Collaborator Author

Most of these issues were addressed in 0.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant