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.

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

Memory leak in Import/Export Plugin #4060

Closed
akhenry opened this issue Jul 27, 2021 · 2 comments · Fixed by #4061
Closed

Memory leak in Import/Export Plugin #4060

akhenry opened this issue Jul 27, 2021 · 2 comments · Fixed by #4061
Labels
performance impacts or improves performance severity:blocker type:bug

Comments

@akhenry
Copy link
Contributor

akhenry commented Jul 27, 2021

Summary

Import/Export plugin holds reference to Open MCT object instance after application has been unloaded, preventing garbage collection.

Expected vs Current Behavior

Import/Export plugin should not prevent garbage collection of Open MCT instance object.

Impact Check List

No user impact. Test only.

Environment

  • Open MCT Version: c3ebf52
  • Deployment Type: Local dev
  • OS: Mac OS X 10.15.7
  • Browser: Chrome headless (automated tests)
@akhenry akhenry mentioned this issue Jul 27, 2021
Merged
7 tasks
@akhenry
Copy link
Contributor Author

akhenry commented Jul 27, 2021

Testing instructions:
Run the automated tests, ensure that they do not fail on the Import/export plugin tests.

@unlikelyzero unlikelyzero added performance impacts or improves performance severity:blocker labels Jul 28, 2021
@unlikelyzero unlikelyzero mentioned this issue Jul 28, 2021
Open
@shefalijoshi
Copy link
Contributor

Tests did not fail... so fixed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
performance impacts or improves performance severity:blocker type:bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed memory leak in import-export plugin nasa/openmct
4 participants
@davetsay @akhenry @shefalijoshi @unlikelyzero