We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
With #7218 we learned that creating more vaults grows the heap. I.e. some objects are being retained through the vault creation that shouldn't be.
Find all the retaining paths and fix them. So far,
The content you are editing has changed. Please copy your edits and refresh the page.
pipeTopicToStorage
Then add regression tests (see Test Plan).
We need a test that checks memory usage growth. This may be achievable by process.memoryUsage() comparison after vault runs. See #7296 (comment)
process.memoryUsage()
The text was updated successfully, but these errors were encountered:
forEachPublicationRecord
turadg
mhofman
No branches or pull requests
What is the Problem Being Solved?
With #7218 we learned that creating more vaults grows the heap. I.e. some objects are being retained through the vault creation that shouldn't be.
Description of the Design
Find all the retaining paths and fix them. So far,
Retention fixes
pipeTopicToStorage
#7303Then add regression tests (see Test Plan).
Security Considerations
Scaling Considerations
Test Plan
We need a test that checks memory usage growth. This may be achievable by
process.memoryUsage()
comparison after vault runs. See #7296 (comment)The text was updated successfully, but these errors were encountered: