hashes are ordered, so just use the columns_hash ivar

1 files changed, 3 insertions, 3 deletions

diff --git a/activerecord/lib/active_record/connection_adapters/abstract/schema_definitions.rb b/activerecord/lib/active_record/connection_adapters/abstract/schema_definitions.rb
index d3abc88fd5..18ddc3c322 100644
--- a/activerecord/lib/active_record/connection_adapters/abstract/schema_definitions.rb
+++ b/activerecord/lib/active_record/connection_adapters/abstract/schema_definitions.rb
@@ -63,15 +63,16 @@ module ActiveRecord
     class TableDefinition
       # An array of ColumnDefinition objects, representing the column changes
       # that have been defined.
-      attr_accessor :columns, :indexes
+      attr_accessor :indexes
 
       def initialize(base)
-        @columns = []
         @columns_hash = {}
         @indexes = {}
         @base = base
       end
 
+      def columns; @columns_hash.values; end
+
       def xml(*args)
         raise NotImplementedError unless %w{
           sqlite mysql mysql2
@@ -301,7 +302,6 @@ module ActiveRecord
 
       def new_column_definition(base, name, type)
         definition = create_column_definition base, name, type
-        @columns << definition
         @columns_hash[name] = definition
         definition
       end