diff options
author | Cody Cutrer <cody@cutrer.us> | 2014-01-21 16:56:53 -0700 |
---|---|---|
committer | Steve Rice <steve@pagerduty.com> | 2014-03-25 21:29:14 -0700 |
commit | afa148a4f0ac5e2a446b5fe87881a130e8a24f3d (patch) | |
tree | f7c274ecbc55c280e48d1355fe217e0f6a9e1585 /activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb | |
parent | 470108b6e95a32e017ba3790d5701d8c8e91bf29 (diff) | |
download | rails-afa148a4f0ac5e2a446b5fe87881a130e8a24f3d.tar.gz rails-afa148a4f0ac5e2a446b5fe87881a130e8a24f3d.tar.bz2 rails-afa148a4f0ac5e2a446b5fe87881a130e8a24f3d.zip |
create indexes inline in CREATE TABLE for MySQL
This is important, because adding an index on a temporary table after
it has been created would commit the transaction
Conflicts:
activerecord/CHANGELOG.md
Diffstat (limited to 'activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb')
-rw-r--r-- | activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb b/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb index 9a819720f7..96c7d5d7eb 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb @@ -6,6 +6,17 @@ module ActiveRecord include Savepoints class SchemaCreation < AbstractAdapter::SchemaCreation + def visit_TableDefinition(o) + create_sql = "CREATE#{' TEMPORARY' if o.temporary} TABLE " + create_sql << "#{quote_table_name(o.name)} " + statements = [] + statements.concat(o.columns.map { |c| accept c }) + statements.concat(o.indexes.map { |(column_name, options)| index_in_create(o.name, column_name, options) }) + create_sql << "(#{statements.join(', ')}) " + create_sql << "#{o.options}" + create_sql << " AS #{@conn.to_sql(o.as)}" if o.as + create_sql + end def visit_AddColumn(o) add_column_position!(super, column_options(o)) @@ -29,6 +40,11 @@ module ActiveRecord end sql end + + def index_in_create(table_name, column_name, options) + index_name, index_type, index_columns, index_options, index_algorithm, index_using = @conn.send(:add_index_options, table_name, column_name, options) + "#{index_type} INDEX #{quote_column_name(index_name)} #{index_using} (#{index_columns})#{index_options} #{index_algorithm}".gsub(' ', ' ').strip + end end def schema_creation @@ -225,6 +241,10 @@ module ActiveRecord version[0] >= 5 end + def supports_indexes_in_create? + true + end + def native_database_types NATIVE_DATABASE_TYPES end |