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

[Task]: Implement LRU caching to cache and evict mock mongoDB connections #36474

Closed
NilanshBansal opened this issue Sep 23, 2024 · 0 comments · Fixed by #36480
Closed

[Task]: Implement LRU caching to cache and evict mock mongoDB connections #36474

NilanshBansal opened this issue Sep 23, 2024 · 0 comments · Fixed by #36480
Assignees
@NilanshBansal
Labels
Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. Integrations Pod Integrations Product Issues related to a specific integration Query & JS Pod Issues related to the query & JS Pod Task A simple Todo

Comments

@NilanshBansal
Copy link
Contributor

SubTasks

Reason: We have recently seen a surge in the number of connections to the mongo movies mockDB leading to the max limit reached.
This directly impacted the cypress tests and the usage of the mockDB for the release, prod and self hosted instances.

On debugging we found the mockDB connections were not getting closed, hence to resolve this we need to implement the LRU caching for the mockDB connections and close the least recently used connections.

Ref: https://theappsmith.slack.com/archives/C02MUD8DNUR/p1726855216593569?thread_ts=1725892395.532789&cid=C02MUD8DNUR
@NilanshBansal NilanshBansal added Task A simple Todo Query & JS Pod Issues related to the query & JS Pod Integrations Product Issues related to a specific integration Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. Integrations Pod labels Sep 23, 2024
@NilanshBansal NilanshBansal self-assigned this Sep 23, 2024
@NilanshBansal NilanshBansal mentioned this issue Sep 23, 2024
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NilanshBansal NilanshBansal

Labels
Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. Integrations Pod Integrations Product Issues related to a specific integration Query & JS Pod Issues related to the query & JS Pod Task A simple Todo
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: added LRU cache for mockDB connections appsmithorg/appsmith
1 participant
@NilanshBansal