Skip to content
Snippets Groups Projects

Cache ES enabled namespaces and projects

Merged Dmitry Gruzd requested to merge 209854-cache-es-check into master

What does this MR do?

A recent postgres-checkup report includes:

SELECT query involving elasticsearch_indexed_projects table consumes >40% of the master resources. The query itself is not slow (13.224 ms/call), but it happens almost 100 times per second.

This MR implements caching of elasticsearch_indexed_projects and elasticsearch_indexed_namespaces.

This cache has a relatively small number of ids and it rarely changes.

#209854 (closed)

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading