Skip to content

GitLab Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
GitLab FOSS
GitLab FOSS
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
    • Iterations
  • Merge Requests 0
    • Merge Requests 0
  • Requirements
    • Requirements
    • List
  • Security & Compliance
    • Security & Compliance
    • Dependency List
    • License Compliance
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Code Review
    • Insights
    • Issue
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
  • GitLab.org
  • GitLab FOSSGitLab FOSS
  • Issues
  • #36877

Closed
Open
Opened Aug 23, 2017 by Yorick Peterse@yorickpeterseMaintainer1 of 1 task completed1/1 task

Reduce SQL timings in Projects::MergeRequestsController#ci_environments_status.json

The controller Projects::MergeRequestsController#ci_environments_status.json is one of our most frequently requested controllers with a 99th percentile (of SQL query timings) above 200 milliseconds. According to our daily overview the statistics are as follows:

  • 401 000 requests per 12 hours
  • 58.8% of the request time is spent in SQL

SQL timings per request:

  • Mean: 50.93 ms
  • 95th percentile: 136.97 ms
  • 99th percentile: 717.06 ms

This controller should be improved so that the 99th percentile is consistently below 200 milliseconds, with an ideal target of 100 - 150 milliseconds.

TODO

  • Remove the index yorick_deployments_test once our query improvements have been deployed
Edited Sep 18, 2017 by Yorick Peterse
Assignee
Assign to
10.0
Milestone
10.0 (Past due)
Assign milestone
Time tracking
None
Due date
None
Reference: gitlab-org/gitlab-foss#36877