Commit 41edd1e3 authored by Rémy Coutable's avatar Rémy Coutable

Merge branch 'rails5-fix-migration-update-manager' into 'master'

Rails5: fix Arel::UpdateManager in MigrateOldElasticsearchSettings migration

See merge request gitlab-org/gitlab-ee!6815
parents ffceb2c6 290a51d2
---
title: 'Rails5: fix Arel::UpdateManager in MigrateOldElasticsearchSettings migration'
merge_request: 6815
author: Jasper Maes
type: fixed
class MigrateOldElasticsearchSettings < ActiveRecord::Migration
include Gitlab::Database::ArelMethods
include Gitlab::Database::MigrationHelpers
DOWNTIME = false
......@@ -23,7 +24,7 @@ class MigrateOldElasticsearchSettings < ActiveRecord::Migration
end
updater =
Arel::UpdateManager.new(ActiveRecord::Base)
arel_update_manager
.table(settings)
.set(settings[:elasticsearch_url] => urls.join(','))
.where(settings[:id].eq(id))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment