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

[Searchable Snapshot] Add Integ test to make there is no resource leak #5244

Closed
Tracked by #5087
aabukhalil opened this issue Nov 14, 2022 · 1 comment · Fixed by #6215
Closed
Tracked by #5087

[Searchable Snapshot] Add Integ test to make there is no resource leak #5244

aabukhalil opened this issue Nov 14, 2022 · 1 comment · Fixed by #6215
Assignees
Labels
distributed framework enhancement Enhancement or improvement to existing feature or request

Comments

@aabukhalil
Copy link
Contributor

Searchable Snapshot and how IndexInput is being used, it has lot of clones, closings and many files handling logic. If we missing closing any file then we will cause resource leaks. We Integ test to make sure there is no resource leak. This is possible (theoretically) by creating remote backed index, do some searches, and then delete the index. And monitor the file descriptor held by process affecting the caching directory.

@aabukhalil
Copy link
Contributor Author

this is estimated as 3 points

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
distributed framework enhancement Enhancement or improvement to existing feature or request
Projects
Status: Done
3 participants