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

sessionList is very slow v7.5 on a cluster #4693

Open
stroomdev10 opened this issue Jan 13, 2025 · 2 comments
Open

sessionList is very slow v7.5 on a cluster #4693

stroomdev10 opened this issue Jan 13, 2025 · 2 comments
Assignees
@at055612
Labels
bug
Milestone
v7.5

Comments

@stroomdev10
Copy link
Member

No description provided.

@stroomdev10 stroomdev10 added this to the v7.5 milestone Jan 13, 2025
@at055612
Copy link
Member

The UI node makes a call out to all nodes to ask them for their session list. If there is a delay in contacting one node it will delay the results for all. It could potentially be changed so that the UI node builds up the cluster session list asynchronously and in the background so that subsequent requests to the servlet read from this if not out of date. The first call to sessionList would need to wait a reasonable time to try to get all nodes' data.

@at055612
Copy link
Member

Issue seems to be that sessions are not expiring so sessionDestroyed event is not fired and thus the in-mem map of sessions is building up. Possibly caused by an uplift to the Jetty version (9->11) when dropwiz was uplifted.

@at055612 at055612 self-assigned this Jan 15, 2025
at055612 added a commit that referenced this issue Jan 15, 2025
@at055612
Empty initial commit on gh-4693-session-leak [no ci]
10cc5a7
at055612 added a commit that referenced this issue Jan 15, 2025
@at055612
gh-4693 Add session list debug logging
ea64c6b 
(cherry picked from commit ee36f06)
at055612 added a commit that referenced this issue Jan 15, 2025
@at055612
gh-4693 Add maxInactiveInterval to control session expiry
21d5e01
@at055612 at055612 mentioned this issue Jan 15, 2025
Merged
at055612 added a commit that referenced this issue Jan 15, 2025
@at055612
Merge pull request #4699 from gchq/gh-4693-session-leak
9c46f4a 
PR for #4693 - sessionList is very slow v7.5 on a cluster
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@at055612 at055612

Labels
bug
Projects
None yet
Milestone
v7.5
Development

No branches or pull requests
2 participants
@at055612 @stroomdev10