diff options
| author | Yorick Peterse <yorickpeterse@gmail.com> | 2017-12-12 18:05:46 +0000 |
|---|---|---|
| committer | Yorick Peterse <yorickpeterse@gmail.com> | 2017-12-12 18:05:46 +0000 |
| commit | 49a9de110e4d2a70d006759529822b3823b63f96 (patch) | |
| tree | f61ecf9868804f871736b827f8cc00bc915d8cf3 | |
| parent | ad1c186ed386afab578668ccdc21cb02354c81bc (diff) | |
| parent | 946d87d332232e1b4bceeac44ce89d8fd645166e (diff) | |
| download | gitlab-ce-49a9de110e4d2a70d006759529822b3823b63f96.tar.gz | |
Merge branch 'optimize-projects-for-imported-projects' into 'master'
check the import_status field before doing SQL operations to check the import url
See merge request gitlab-org/gitlab-ce!15752
| -rw-r--r-- | app/models/project.rb | 3 | ||||
| -rw-r--r-- | changelogs/unreleased/optimize-projects-for-imported-projects.yml | 6 |
2 files changed, 8 insertions, 1 deletions
diff --git a/app/models/project.rb b/app/models/project.rb index 4c4ec2226ad..5183a216c53 100644 --- a/app/models/project.rb +++ b/app/models/project.rb @@ -659,7 +659,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 |