Merge pull request rails#33977 from marshall-lee/abandon-top-support

Abandon TOP support.

Author: kamipo

activerecord/lib/arel/nodes/select_core.rb

@@ -3,13 +3,12 @@
 module Arel # :nodoc: all
   module Nodes
     class SelectCore < Arel::Nodes::Node
-      attr_accessor :top, :projections, :wheres, :groups, :windows
+      attr_accessor :projections, :wheres, :groups, :windows
       attr_accessor :havings, :source, :set_quantifier
 
       def initialize
         super()
         @source         = JoinSource.new nil
-        @top            = nil
 
         # https://ronsavage.github.io/SQL/sql-92.bnf.html#set%20quantifier
         @set_quantifier = nil
@@ -43,15 +42,14 @@ def initialize_copy(other)
 
       def hash
         [
-          @source, @top, @set_quantifier, @projections,
+          @source, @set_quantifier, @projections,
           @wheres, @groups, @havings, @windows
         ].hash
       end
 
       def eql?(other)
         self.class == other.class &&
           self.source == other.source &&
-          self.top == other.top &&
           self.set_quantifier == other.set_quantifier &&
           self.projections == other.projections &&
           self.wheres == other.wheres &&

activerecord/lib/arel/nodes/unary.rb

@@ -37,7 +37,6 @@ def eql?(other)
       On
       Ordering
       RollUp
-      Top
     }.each do |name|
       const_set(name, Class.new(Unary))
     end

activerecord/lib/arel/select_manager.rb

@@ -222,10 +222,8 @@ def with(*subqueries)
     def take(limit)
       if limit
         @ast.limit = Nodes::Limit.new(limit)
-        @ctx.top   = Nodes::Top.new(limit)
       else
         @ast.limit = nil
-        @ctx.top   = nil
       end
       self
     end

activerecord/lib/arel/visitors/depth_first.rb

@@ -34,7 +34,6 @@ def unary(o)
         alias :visit_Arel_Nodes_Ordering          :unary
         alias :visit_Arel_Nodes_Ascending         :unary
         alias :visit_Arel_Nodes_Descending        :unary
-        alias :visit_Arel_Nodes_Top               :unary
         alias :visit_Arel_Nodes_UnqualifiedColumn :unary
 
         def function(o)

activerecord/lib/arel/visitors/dot.rb

@@ -81,7 +81,6 @@ def unary(o)
         alias :visit_Arel_Nodes_Not               :unary
         alias :visit_Arel_Nodes_Offset            :unary
         alias :visit_Arel_Nodes_On                :unary
-        alias :visit_Arel_Nodes_Top               :unary
         alias :visit_Arel_Nodes_UnqualifiedColumn :unary
         alias :visit_Arel_Nodes_Preceding         :unary
         alias :visit_Arel_Nodes_Following         :unary

activerecord/lib/arel/visitors/mssql.rb

@@ -12,13 +12,6 @@ def initialize(*)
 
       private
 
-        # `top` wouldn't really work here. I.e. User.select("distinct first_name").limit(10) would generate
-        # "select top 10 distinct first_name from users", which is invalid query! it should be
-        # "select distinct top 10 first_name from users"
-        def visit_Arel_Nodes_Top(o)
-          ""
-        end
-
         def visit_Arel_Visitors_MSSQL_RowNumber(o, collector)
           collector << "ROW_NUMBER() OVER (ORDER BY "
           inject_join(o.children, collector, ", ") << ") as _row_num"

activerecord/lib/arel/visitors/to_sql.rb

@@ -237,8 +237,6 @@ def visit_Arel_Nodes_SelectOptions(o, collector)
         def visit_Arel_Nodes_SelectCore(o, collector)
           collector << "SELECT"
 
-          collector = maybe_visit o.top, collector
-
           collector = maybe_visit o.set_quantifier, collector
 
           collect_nodes_for o.projections, collector, SPACE
@@ -405,11 +403,6 @@ def visit_Arel_Nodes_Limit(o, collector)
           visit o.expr, collector
         end
 
-        # FIXME: this does nothing on most databases, but does on MSSQL
-        def visit_Arel_Nodes_Top(o, collector)
-          collector
-        end
-
         def visit_Arel_Nodes_Lock(o, collector)
           visit o.expr, collector
         end

activerecord/test/cases/arel/select_manager_test.rb

@@ -131,15 +131,15 @@ def test_join_sources
           right = table.alias
           mgr   = table.from
           mgr.join(right).on("omg")
-          mgr.to_sql.must_be_like %{ SELECT  FROM "users" INNER JOIN "users" "users_2" ON omg }
+          mgr.to_sql.must_be_like %{ SELECT FROM "users" INNER JOIN "users" "users_2" ON omg }
         end
 
         it "converts to sqlliterals with multiple items" do
           table = Table.new :users
           right = table.alias
           mgr   = table.from
           mgr.join(right).on("omg", "123")
-          mgr.to_sql.must_be_like %{ SELECT  FROM "users" INNER JOIN "users" "users_2" ON omg AND 123 }
+          mgr.to_sql.must_be_like %{ SELECT FROM "users" INNER JOIN "users" "users_2" ON omg AND 123 }
         end
       end
     end

activerecord/test/cases/arel/visitors/depth_first_test.rb

@@ -33,7 +33,6 @@ def test_raises_with_object
         Arel::Nodes::Ordering,
         Arel::Nodes::StringJoin,
         Arel::Nodes::UnqualifiedColumn,
-        Arel::Nodes::Top,
         Arel::Nodes::Limit,
         Arel::Nodes::Else,
       ].each do |klass|

activerecord/test/cases/arel/visitors/dot_test.rb

@@ -37,7 +37,6 @@ def test_named_function
         Arel::Nodes::Offset,
         Arel::Nodes::Ordering,
         Arel::Nodes::UnqualifiedColumn,
-        Arel::Nodes::Top,
         Arel::Nodes::Limit,
       ].each do |klass|
         define_method("test_#{klass.name.gsub('::', '_')}") do