Revert unused code and re-using query annotation for `update_all` and `delete_all`

This partly reverts #35617.

#35617 includes unused code (for `InsertStatement`) and re-using query
annotation for `update_all` and `delete_all`, which has not been
discussed yet.

If a relation has any annotation, I think it is mostly for SELECT query,
so re-using annotation by default is not always desired behavior for me.

We should discuss about desired behavior before publishing the
implementation.

1 files changed, 0 insertions, 24 deletions

diff --git a/activerecord/test/cases/arel/update_manager_test.rb b/activerecord/test/cases/arel/update_manager_test.rb
index e13cb6aa52..cc1b9ac5b3 100644
--- a/activerecord/test/cases/arel/update_manager_test.rb
+++ b/activerecord/test/cases/arel/update_manager_test.rb
@@ -122,29 +122,5 @@ module Arel
         @um.key.must_equal @table[:foo]
       end
     end
-
-    describe "comment" do
-      it "chains" do
-        manager = Arel::UpdateManager.new
-        manager.comment("updating").must_equal manager
-      end
-
-      it "appends a comment to the generated query" do
-        table   = Table.new :users
-
-        manager = Arel::UpdateManager.new
-        manager.table table
-
-        manager.comment("updating")
-        manager.to_sql.must_be_like %{
-          UPDATE "users" /* updating */
-        }
-
-        manager.comment("updating", "with", "comment")
-        manager.to_sql.must_be_like %{
-          UPDATE "users" /* updating */ /* with */ /* comment */
-        }
-      end
-    end
   end
 end