diff options
| author | Stan Hu <stanhu@gmail.com> | 2016-11-06 13:54:54 -0800 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2016-11-06 14:00:09 -0800 |
| commit | a8f2fceadd60db759522c5669b99e68029df912e (patch) | |
| tree | 46812b7e07cd513df851a34f72c7969d3b2c39e4 | |
| parent | c392b0cc24ba40e3fed920c6c693cb24665193af (diff) | |
| download | gitlab-ce-add-project-import-data-index.tar.gz | |
Add an index for project_id in project_import_data to improveadd-project-import-data-index
performance
We see that many slow queries on GitLab.com are dominated by finding
the project import data for a specific project. Adding an index is
the most straightforward way of fixing this.
Closes #23748
| -rw-r--r-- | changelogs/unreleased/add-project-import-data-index.yml | 4 | ||||
| -rw-r--r-- | db/migrate/20161106185620_add_project_import_data_project_index.rb | 12 | ||||
| -rw-r--r-- | db/schema.rb | 4 |
3 files changed, 19 insertions, 1 deletions
diff --git a/changelogs/unreleased/add-project-import-data-index.yml b/changelogs/unreleased/add-project-import-data-index.yml new file mode 100644 index 00000000000..f5e4005f544 --- /dev/null +++ b/changelogs/unreleased/add-project-import-data-index.yml @@ -0,0 +1,4 @@ +--- +title: Add an index for project_id in project_import_data to improve performance +merge_request: +author: diff --git a/db/migrate/20161106185620_add_project_import_data_project_index.rb b/db/migrate/20161106185620_add_project_import_data_project_index.rb new file mode 100644 index 00000000000..750a6a8c51e --- /dev/null +++ b/db/migrate/20161106185620_add_project_import_data_project_index.rb @@ -0,0 +1,12 @@ +class AddProjectImportDataProjectIndex < ActiveRecord::Migration + include Gitlab::Database::MigrationHelpers + + # Set this constant to true if this migration requires downtime. + DOWNTIME = false + + disable_ddl_transaction! + + def change + add_concurrent_index :project_import_data, :project_id + end +end diff --git a/db/schema.rb b/db/schema.rb index 5476b0c93e5..212b0ed1c58 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -11,7 +11,7 @@ # # It's strongly recommended that you check this file into your version control system. -ActiveRecord::Schema.define(version: 20161103171205) do +ActiveRecord::Schema.define(version: 20161106185620) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -867,6 +867,8 @@ ActiveRecord::Schema.define(version: 20161103171205) do t.string "encrypted_credentials_salt" end + add_index "project_import_data", ["project_id"], name: "index_project_import_data_on_project_id", using: :btree + create_table "projects", force: :cascade do |t| t.string "name" t.string "path" |