Merge pull request rails#55311 from tarellel/main

Cleanup typos and grammer in database adapter comments

Author: byroot

activerecord/lib/active_record/connection_adapters/abstract/connection_handler.rb

@@ -168,7 +168,7 @@ def clear_active_connections!(role = nil)
         end
       end
 
-      # Clears the cache which maps classes.
+      # Clears reloadable connection caches in all connection pools.
       #
       # See ConnectionPool#clear_reloadable_connections! for details.
       def clear_reloadable_connections!(role = nil)

activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb

@@ -129,7 +129,7 @@ class ConnectionPool
       else
         class WeakThreadKeyMap # :nodoc:
           # FIXME: On 3.3 we could use ObjectSpace::WeakKeyMap
-          # but it currently cause GC crashes: https://github.com/byroot/rails/pull/3
+          # but it currently causes GC crashes: https://github.com/byroot/rails/pull/3
           def initialize
             @map = {}
           end
@@ -518,7 +518,7 @@ def discarded? # :nodoc:
         @connections.nil?
       end
 
-      # Clears the cache which maps classes and re-connects connections that
+      # Clears reloadable connections from the pool and re-connects connections that
       # require reloading.
       #
       # Raises:
@@ -541,7 +541,7 @@ def clear_reloadable_connections(raise_on_acquisition_timeout = true)
         end
       end
 
-      # Clears the cache which maps classes and re-connects connections that
+      # Clears reloadable connections from the pool and re-connects connections that
       # require reloading.
       #
       # The pool first tries to gain ownership of all connections. If unable to

activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb

@@ -612,7 +612,7 @@ def rename_enum_value(type_name, **options)
         }
       end
 
-      # Returns the configured supported identifier length supported by PostgreSQL
+      # Returns the configured maximum supported identifier length supported by PostgreSQL
       def max_identifier_length
         @max_identifier_length ||= query_value("SHOW max_identifier_length", "SCHEMA").to_i
       end

activerecord/lib/active_record/connection_adapters/sqlite3/database_statements.rb

@@ -134,8 +134,8 @@ def perform_query(raw_connection, sql, binds, type_casted_binds, prepare:, notif
           end
 
           def cast_result(result)
-            # Given that SQLite3 doesn't really a Result type, raw_execute already return an ActiveRecord::Result
-            # and we have nothing to cast here.
+            # Given that SQLite3 doesn't have a Result type, raw_execute already returns an ActiveRecord::Result
+            # so we have nothing to cast here.
             result
           end
 

activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rb

@@ -41,7 +41,7 @@ module ConnectionAdapters # :nodoc:
     #
     # There may be other options available specific to the SQLite3 driver. Please read the
     # documentation for
-    # {SQLite::Database.new}[https://sparklemotion.github.io/sqlite3-ruby/SQLite3/Database.html#method-c-new]
+    # {SQLite3::Database.new}[https://sparklemotion.github.io/sqlite3-ruby/SQLite3/Database.html#method-c-new]
     #
     class SQLite3Adapter < AbstractAdapter
       ADAPTER_NAME = "SQLite"
@@ -337,7 +337,7 @@ def virtual_tables
       # Creates a virtual table
       #
       # Example:
-      #   create_virtual_table :emails, :fts5, ['sender', 'title',' body']
+      #   create_virtual_table :emails, :fts5, ['sender', 'title', 'body']
       def create_virtual_table(table_name, module_name, values)
         exec_query "CREATE VIRTUAL TABLE IF NOT EXISTS #{table_name} USING #{module_name} (#{values.join(", ")})"
       end