Improve performance on RemoveDuplicatesFromRoutes migration
This commit is contained in:
parent
fd0d8a2832
commit
ad4ec44673
|
@ -12,20 +12,16 @@ class RemoveDuplicatesFromRoutes < ActiveRecord::Migration
|
||||||
# to fill these values that avoid duplicate entries in the routes table.
|
# to fill these values that avoid duplicate entries in the routes table.
|
||||||
return unless Gitlab::Database.mysql?
|
return unless Gitlab::Database.mysql?
|
||||||
|
|
||||||
select_all("SELECT path FROM #{quote_table_name(:routes)} GROUP BY path HAVING COUNT(*) > 1").each do |row|
|
execute <<-EOF
|
||||||
path = connection.quote(row['path'])
|
DELETE duplicated_rows.*
|
||||||
execute(%Q{
|
FROM routes AS duplicated_rows
|
||||||
DELETE FROM #{quote_table_name(:routes)}
|
INNER JOIN (
|
||||||
WHERE path = #{path}
|
SELECT path, MAX(id) as max_id
|
||||||
AND id != (
|
FROM routes
|
||||||
SELECT id FROM (
|
GROUP BY path
|
||||||
SELECT max(id) AS id
|
HAVING COUNT(*) > 1
|
||||||
FROM #{quote_table_name(:routes)}
|
) AS good_rows ON good_rows.path = duplicated_rows.path AND good_rows.max_id <> duplicated_rows.id;
|
||||||
WHERE path = #{path}
|
EOF
|
||||||
) max_ids
|
|
||||||
)
|
|
||||||
})
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
def down
|
def down
|
||||||
|
|
Loading…
Reference in New Issue