diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2018-09-05 11:17:41 +0000 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2018-09-05 11:17:41 +0000 |
commit | 4a3a74979796e6245bd82dd06c52146caecb9b9e (patch) | |
tree | df743dd84b60e4280d8ee8db4b64b7a462a6ddf3 | |
parent | df1a9d67411d51291b97c5e62a31653675fe63ec (diff) | |
parent | d5e9185618cc30227b2bd68f4322a3d7860152ee (diff) | |
download | gitlab-ce-4a3a74979796e6245bd82dd06c52146caecb9b9e.tar.gz |
Merge branch 'fix_mysql_binary_index_patch' into 'master'
Make MySQL patch for binary indexes compatible with composite indexes
See merge request gitlab-org/gitlab-ce!21511
-rw-r--r-- | config/initializers/mysql_set_length_for_binary_indexes.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/config/initializers/mysql_set_length_for_binary_indexes.rb b/config/initializers/mysql_set_length_for_binary_indexes.rb index 1b16b39d517..81ed2fb83de 100644 --- a/config/initializers/mysql_set_length_for_binary_indexes.rb +++ b/config/initializers/mysql_set_length_for_binary_indexes.rb @@ -7,11 +7,12 @@ # the old Rails 4 schema layout is still used module MysqlSetLengthForBinaryIndex def add_index(table_name, column_names, options = {}) + options[:length] ||= {} Array(column_names).each do |column_name| column = ActiveRecord::Base.connection.columns(table_name).find { |c| c.name == column_name } if column&.type == :binary - options[:length] = 20 + options[:length][column_name] = 20 end end @@ -27,11 +28,12 @@ if Gitlab.rails5? module MysqlSetLengthForBinaryIndexAndIgnorePostgresOptionsForSchema # This method is used in Rails 5 schema loading as t.index def index(column_names, options = {}) + options[:length] ||= {} Array(column_names).each do |column_name| column = columns.find { |c| c.name == column_name } if column&.type == :binary - options[:length] = 20 + options[:length][column_name] = 20 end end |