fixup! Lfs pruning improvements and review changes

3 files changed, 0 insertions, 63 deletions

diff --git a/db/migrate/20170922164114_create_unprocessed_lfs_pushes.rb b/db/migrate/20170922164114_create_unprocessed_lfs_pushes.rb
index 63d2b59d22d..e293b50345a 100644
--- a/db/migrate/20170922164114_create_unprocessed_lfs_pushes.rb
+++ b/db/migrate/20170922164114_create_unprocessed_lfs_pushes.rb
@@ -1,30 +1,9 @@
-# See http://doc.gitlab.com/ce/development/migration_style_guide.html
-# for more information on how to write migrations for GitLab.
-
 class CreateUnprocessedLfsPushes < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
 
   # Set this constant to true if this migration requires downtime.
   DOWNTIME = false
 
-  # When a migration requires downtime you **must** uncomment the following
-  # constant and define a short and easy to understand explanation as to why the
-  # migration requires downtime.
-  # DOWNTIME_REASON = ''
-
-  # When using the methods "add_concurrent_index", "remove_concurrent_index" or
-  # "add_column_with_default" you must disable the use of transactions
-  # as these methods can not run in an existing transaction.
-  # When using "add_concurrent_index" or "remove_concurrent_index" methods make sure
-  # that either of them is the _only_ method called in the migration,
-  # any other changes should go in a separate migration.
-  # This ensures that upon failure _only_ the index creation or removing fails
-  # and can be retried or reverted easily.
-  #
-  # To disable transactions uncomment the following line and remove these
-  # comments:
-  # disable_ddl_transaction!
-
   def change
     create_table :unprocessed_lfs_pushes do |t|
       t.references :project, null: false, index: true, foreign_key: { on_delete: :cascade }
diff --git a/db/migrate/20170922171356_create_lfs_pointer.rb b/db/migrate/20170922171356_create_lfs_pointer.rb
index 95232f0b110..6085c7a8a48 100644
--- a/db/migrate/20170922171356_create_lfs_pointer.rb
+++ b/db/migrate/20170922171356_create_lfs_pointer.rb
@@ -1,30 +1,9 @@
-# See http://doc.gitlab.com/ce/development/migration_style_guide.html
-# for more information on how to write migrations for GitLab.
-
 class CreateLfsPointer < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
 
   # Set this constant to true if this migration requires downtime.
   DOWNTIME = false
 
-  # When a migration requires downtime you **must** uncomment the following
-  # constant and define a short and easy to understand explanation as to why the
-  # migration requires downtime.
-  # DOWNTIME_REASON = ''
-
-  # When using the methods "add_concurrent_index", "remove_concurrent_index" or
-  # "add_column_with_default" you must disable the use of transactions
-  # as these methods can not run in an existing transaction.
-  # When using "add_concurrent_index" or "remove_concurrent_index" methods make sure
-  # that either of them is the _only_ method called in the migration,
-  # any other changes should go in a separate migration.
-  # This ensures that upon failure _only_ the index creation or removing fails
-  # and can be retried or reverted easily.
-  #
-  # To disable transactions uncomment the following line and remove these
-  # comments:
-  # disable_ddl_transaction!
-
   def change
     create_table :lfs_pointers do |t|
       t.references :project, null: false, index: true, foreign_key: { on_delete: :cascade }
diff --git a/db/migrate/20171013174017_create_processed_lfs_refs.rb b/db/migrate/20171013174017_create_processed_lfs_refs.rb
index f149ae00d97..2241feaae0e 100644
--- a/db/migrate/20171013174017_create_processed_lfs_refs.rb
+++ b/db/migrate/20171013174017_create_processed_lfs_refs.rb
@@ -1,30 +1,9 @@
-# See http://doc.gitlab.com/ce/development/migration_style_guide.html
-# for more information on how to write migrations for GitLab.
-
 class CreateProcessedLfsRefs < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
 
   # Set this constant to true if this migration requires downtime.
   DOWNTIME = false
 
-  # When a migration requires downtime you **must** uncomment the following
-  # constant and define a short and easy to understand explanation as to why the
-  # migration requires downtime.
-  # DOWNTIME_REASON = ''
-
-  # When using the methods "add_concurrent_index", "remove_concurrent_index" or
-  # "add_column_with_default" you must disable the use of transactions
-  # as these methods can not run in an existing transaction.
-  # When using "add_concurrent_index" or "remove_concurrent_index" methods make sure
-  # that either of them is the _only_ method called in the migration,
-  # any other changes should go in a separate migration.
-  # This ensures that upon failure _only_ the index creation or removing fails
-  # and can be retried or reverted easily.
-  #
-  # To disable transactions uncomment the following line and remove these
-  # comments:
-  # disable_ddl_transaction!
-
   def change
     create_table :processed_lfs_refs do |t|
       t.references :project, null: false, index: true, foreign_key: { on_delete: :cascade }