diff options
| author | Kamil Trzciński <ayufan@ayufan.eu> | 2019-06-19 12:18:04 +0300 |
|---|---|---|
| committer | Kamil Trzciński <ayufan@ayufan.eu> | 2019-06-19 13:23:05 +0300 |
| commit | 3f543cd2c93d987723d51d629000b5550eb59636 (patch) | |
| tree | f39d5b2261ee9b3cddc8ab9ce9d35eb0b4a95fae /app/controllers/projects/jobs_controller.rb | |
| parent | 8b55aaee33403c60fefe53a7e58a398c50019388 (diff) | |
Fix N+1 problem in `JobsController#index`
This adds missing preloads, and introduces
additional n+1 matcher to look for
duplicates.
Diffstat (limited to 'app/controllers/projects/jobs_controller.rb')
| -rw-r--r-- | app/controllers/projects/jobs_controller.rb | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/app/controllers/projects/jobs_controller.rb b/app/controllers/projects/jobs_controller.rb index 2a4933e7bc2..d7c0039b234 100644 --- a/app/controllers/projects/jobs_controller.rb +++ b/app/controllers/projects/jobs_controller.rb @@ -31,8 +31,12 @@ class Projects::JobsController < Projects::ApplicationController @builds end @builds = @builds.includes([ - { pipeline: :project }, + { pipeline: [:project, :user] }, + :job_artifacts_archive, + :metadata, + :trigger_request, :project, + :user, :tags ]) @builds = @builds.page(params[:page]).per(30).without_count |