From 10e00ea0457bf7fc9cfba00c8126cca63ebf0730 Mon Sep 17 00:00:00 2001 From: Greg Stark Date: Tue, 5 Dec 2017 19:26:02 +0000 Subject: check the import_status field before doing SQL operations to check the import url --- app/models/project.rb | 3 ++- changelogs/unreleased/optimize-projects-for-imported-projects.yml | 6 ++++++ 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 changelogs/unreleased/optimize-projects-for-imported-projects.yml diff --git a/app/models/project.rb b/app/models/project.rb index cc530076bf7..e65c5290699 100644 --- a/app/models/project.rb +++ b/app/models/project.rb @@ -657,7 +657,8 @@ class Project < ActiveRecord::Base end def import_started? - import? && import_status == 'started' + # import? does SQL work so only run it if it looks like there's an import running + import_status == 'started' && import? end def import_scheduled? diff --git a/changelogs/unreleased/optimize-projects-for-imported-projects.yml b/changelogs/unreleased/optimize-projects-for-imported-projects.yml new file mode 100644 index 00000000000..13186fa36d5 --- /dev/null +++ b/changelogs/unreleased/optimize-projects-for-imported-projects.yml @@ -0,0 +1,6 @@ +--- +title: check the import_status field before doing SQL operations to check the import + url +merge_request: +author: +type: performance -- cgit v1.2.1