Revert "Merge pull request #13344 from ccutrer/fix-from-default-select"

This reverts commit 3ea8403554, reversing
changes made to e4cde5d58c.

Conflicts:
	activerecord/CHANGELOG.md
	activerecord/lib/active_record/relation/query_methods.rb

Reason: using `from` without `select` should not change the select list
to SELECT * because it can lead different query results. If it is needed
to change the table to a subquery or a view you can pass a table alias
in the `from` call or use `select('subquery.*')`.

Fixes #14049.

activerecord/CHANGELOG.md

@@ -533,22 +533,6 @@
 
     *Damien Mathieu*
 
-*   Improve the default select when `from` is used.
-
-    Previously, if you did something like Topic.from(:temp_topics), it
-    would generate SQL like:
-
-        SELECT topics.* FROM temp_topics;
-
-    Which is will cause an error since there's not a topics table to select
-    from.
-
-    Now the default if you use from is just `*`:
-
-        SELECT * FROM temp_topics;
-
-    *Cody Cutrer*
-
 *   Fix `PostgreSQL` insert to properly extract table name from multiline string SQL.
 
     Previously, executing an insert SQL in `PostgreSQL` with a command like this:

activerecord/lib/active_record/relation/query_methods.rb

@@ -995,8 +995,6 @@ module ActiveRecord
           columns_hash.key?(field.to_s) ? arel_table[field] : field
         end
         arel.project(*expanded_select)
-      elsif from_value
-        arel.project(Arel.star)
       else
         arel.project(@klass.arel_table[Arel.star])
       end

activerecord/test/cases/relations_test.rb

@@ -151,11 +151,14 @@ class RelationTest < ActiveRecord::TestCase
     assert_equal relation.to_a, Comment.select('a.*').from(relation, :a).to_a
   end
 
-  def test_finding_with_subquery_without_select
+  def test_finding_with_subquery_without_select_does_not_change_the_select
-    relation = Topic.where(:approved => true)
+    relation = Topic.where(approved: true)
-    assert_equal relation.to_a, Topic.from(relation).to_a
+    assert_raises(ActiveRecord::StatementInvalid) do
+      Topic.from(relation).to_a
+    end
   end
 
+
   def test_finding_with_conditions
     assert_equal ["David"], Author.where(:name => 'David').map(&:name)
     assert_equal ['Mary'],  Author.where(["name = ?", 'Mary']).map(&:name)