Fixed comments for ActiveRecord 5.2.x methods

Author: keithdoggett

lib/active_record/connection_adapters/cockroachdb/quoting.rb

@@ -30,15 +30,6 @@ def _quote(value)
           else
             super
           end
-          # if value.is_a?(Numeric)
-          #   "'#{quote_string(value.to_s)}'"
-          # elsif RGeo::Feature::Geometry.check_type(value)
-          #   "'#{RGeo::WKRep::WKBGenerator.new(hex_format: true, type_format: :ewkb, emit_ewkb_srid: true).generate(value)}'"
-          # elsif value.is_a?(RGeo::Cartesian::BoundingBox)
-          #   "'#{value.min_x},#{value.min_y},#{value.max_x},#{value.max_y}'::box"
-          # else
-          #   super
-          # end
         end
       end
     end

lib/active_record/connection_adapters/cockroachdb_adapter.rb

@@ -75,11 +75,9 @@ class CockroachDBAdapter < PostgreSQLAdapter
       # override
       # This method makes a sql query to gather information about columns
       # in a table. It returns an array of arrays (one for each col) and
-      # passes each to the SchemaStatements#new_column_from_field method
-      # as the field parameter. This data is then used to format the column
-      # objects for the model and sent to the OID for data casting.
+      # is mapped to columns in the SchemaStatements#columns method.
       #
-      # The issue with the default method is that the sql_type field is 
+      # The issue with the default method is that the sql_type field is
       # retrieved with the `format_type` function, but this is implemented
       # differently in CockroachDB than PostGIS, so geometry/geography
       # types are missing information which makes parsing them impossible.
@@ -92,7 +90,7 @@ class CockroachDBAdapter < PostgreSQLAdapter
       #
       # The solution is to make the default query with super, then
       # iterate through the columns and if it is a spatial type,
-      # access the proper column_type with the information_schema.columns 
+      # access the proper column_type with the information_schema.columns
       # table.
       #
       # @see: https://github.com/rails/rails/blob/8695b028261bdd244e254993255c6641bdbc17a5/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb#L829