diff options
| author | Greg Stark <stark@gitlab.com> | 2018-01-11 13:22:52 +0000 |
|---|---|---|
| committer | Greg Stark <stark@gitlab.com> | 2018-01-11 17:45:42 +0000 |
| commit | b64b7d70038fcf38495f67bc2f9cb320c2c168e7 (patch) | |
| tree | 3c27fcf014677507de71c08cb0ddaee94de65357 | |
| parent | 2f585071cba37b2b85f3148460d98c415a1df2d8 (diff) | |
| download | gitlab-ce-disable-pages-on-jobs.tar.gz | |
Use simple Next/Prev paging for jobs to avoid large count queries on arbitrarily large sets of historical jobsdisable-pages-on-jobs
| -rw-r--r-- | app/controllers/projects/jobs_controller.rb | 2 | ||||
| -rw-r--r-- | app/views/projects/jobs/_table.html.haml | 2 | ||||
| -rw-r--r-- | changelogs/unreleased/disable-pages-on-jobs.yml | 6 |
3 files changed, 8 insertions, 2 deletions
diff --git a/app/controllers/projects/jobs_controller.rb b/app/controllers/projects/jobs_controller.rb index 4865ec3dfe5..8b54ba3ad7c 100644 --- a/app/controllers/projects/jobs_controller.rb +++ b/app/controllers/projects/jobs_controller.rb @@ -29,7 +29,7 @@ class Projects::JobsController < Projects::ApplicationController :project, :tags ]) - @builds = @builds.page(params[:page]).per(30) + @builds = @builds.page(params[:page]).per(30).without_count end def cancel_all diff --git a/app/views/projects/jobs/_table.html.haml b/app/views/projects/jobs/_table.html.haml index 82806f022ee..d124d3ebfc1 100644 --- a/app/views/projects/jobs/_table.html.haml +++ b/app/views/projects/jobs/_table.html.haml @@ -22,4 +22,4 @@ = render partial: "projects/ci/builds/build", collection: builds, as: :build, locals: { commit_sha: true, ref: true, pipeline_link: true, stage: true, allow_retry: true, admin: admin } - = paginate builds, theme: 'gitlab' + = paginate_collection(builds) diff --git a/changelogs/unreleased/disable-pages-on-jobs.yml b/changelogs/unreleased/disable-pages-on-jobs.yml new file mode 100644 index 00000000000..629768efce1 --- /dev/null +++ b/changelogs/unreleased/disable-pages-on-jobs.yml @@ -0,0 +1,6 @@ +--- +title: Use simple Next/Prev paging for jobs to avoid large count queries on arbitrarily + large sets of historical jobs +merge_request: +author: +type: performance |