activerecord-jdbc-alt-adapter 52.5.1-java → 60.2.0-java

data/lib/arjdbc/abstract/statement_cache.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_record/connection_adapters/statement_pool'
 
 module ArJdbc
@@ -41,11 +43,6 @@ module ArJdbc
         @statements[sql_key(sql)] ||= @connection.prepare_statement(sql)
       end
 
-      def supports_statement_cache?
-        ActiveSupport::Deprecation.deprecation_warning(__method__)
-        @jdbc_statement_cache_enabled
-      end
-
       private
 
       # This should be overridden by the adapter if the sql itself

data/lib/arjdbc/abstract/transaction_support.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArJdbc
   module Abstract
 
@@ -22,7 +24,7 @@ module ArJdbc
       # Starts a database transaction.
       # @override
       def begin_db_transaction
-        log('BEGIN TRANSACTION'.freeze, nil) { @connection.begin }
+        log('BEGIN TRANSACTION', nil) { @connection.begin }
       end
 
       # Starts a database transaction.
@@ -34,14 +36,14 @@ module ArJdbc
       # Commits the current database transaction.
       # @override
       def commit_db_transaction
-        log('COMMIT TRANSACTION'.freeze, nil) { @connection.commit }
+        log('COMMIT TRANSACTION', nil) { @connection.commit }
       end
 
       # Rolls back the current database transaction.
       # Called from 'rollback_db_transaction' in the AbstractAdapter
       # @override
       def exec_rollback_db_transaction
-        log('ROLLBACK TRANSACTION'.freeze, nil) { @connection.rollback }
+        log('ROLLBACK TRANSACTION', nil) { @connection.rollback }
       end
 
       ########################## Savepoint Interface ############################

data/lib/arjdbc/db2/column.rb

@@ -80,45 +80,6 @@ module ArJdbc
 
       private
 
-      def simplified_type(field_type)
-        case field_type
-        when /^decimal\(1\)$/i   then DB2.emulate_booleans? ? :boolean : :integer
-        when /smallint/i         then DB2.emulate_booleans? ? :boolean : :integer
-        when /boolean/i          then :boolean
-        when /^real|double/i     then :float
-        when /int|serial/i       then :integer
-        # if a numeric column has no scale, lets treat it as an integer.
-        # The AS400 rpg guys do this ALOT since they have no integer datatype ...
-        when /decimal|numeric|decfloat/i
-          extract_scale(field_type) == 0 ? :integer : :decimal
-        when /timestamp/i        then :timestamp
-        when /datetime/i         then :datetime
-        when /time/i             then :time
-        when /date/i             then :date
-        # DB2 provides three data types to store these data objects as strings of up to 2 GB in size:
-        #  Character large objects (CLOBs)
-        #    Use the CLOB data type to store SBCS or mixed data, such as documents that contain
-        #    single character set. Use this data type if your data is larger (or might grow larger)
-        #    than the VARCHAR data type permits.
-        #  Double-byte character large objects (DBCLOBs)
-        #    Use the DBCLOB data type to store large amounts of DBCS data, such as documents that
-        #    use a DBCS character set.
-        #  Binary large objects (BLOBs)
-        #    Use the BLOB data type to store large amounts of noncharacter data, such as pictures,
-        #    voice, and mixed media.
-        when /clob|text/i        then :text # handles DBCLOB
-        when /^long varchar/i    then :text # :limit => 32700
-        when /blob|binary/i      then :binary
-        # varchar () for bit data, char () for bit data, long varchar for bit data
-        when /for bit data/i     then :binary
-        when /xml/i              then :xml
-        when /graphic/i          then :graphic # vargraphic, long vargraphic
-        when /rowid/i            then :rowid # rowid is a supported datatype on z/OS and i/5
-        else
-          super
-        end
-      end
-
       # Post process default value from JDBC into a Rails-friendly format (columns{-internal})
       def default_value(value)
         # IBM i (AS400) will return an empty string instead of null for no default

data/lib/arjdbc/derby/adapter.rb

@@ -65,25 +65,6 @@ module ArJdbc
         limit
       end
 
-      def simplified_type(field_type)
-        case field_type
-        when /^smallint/i    then Derby.emulate_booleans? ? :boolean : :integer
-        when /^bigint|int/i  then :integer
-        when /^real|double/i then :float
-        when /^dec/i         then # DEC is a DECIMAL alias
-          extract_scale(field_type) == 0 ? :integer : :decimal
-        when /^timestamp/i   then :datetime
-        when /^xml/i         then :xml
-        when 'long varchar'  then :text
-        when /for bit data/i then :binary
-        # :name=>"long varchar for bit data", :limit=>32700
-        # :name=>"varchar() for bit data", :limit=>32672
-        # :name=>"char() for bit data", :limit=>254}
-        else
-          super
-        end
-      end
-
       # Post process default value from JDBC into a Rails-friendly format (columns{-internal})
       def default_value(value)
         # JDBC returns column default strings with actual single quotes around the value.
@@ -420,7 +401,7 @@ module ArJdbc
 
     # @override
     def supports_ddl_transactions?; true end
- 
+
     # @override
     def supports_foreign_keys?; true end
 

data/lib/arjdbc/firebird/adapter.rb

@@ -38,27 +38,6 @@ module ArJdbc
           return $1 unless $1.upcase == 'NULL'
         end
       end
-
-      private
-
-      def simplified_type(field_type)
-        case field_type
-        when /timestamp/i    then :datetime
-        when /^smallint/i    then :integer
-        when /^bigint|int/i  then :integer
-        when /^double/i      then :float # double precision
-        when /^decimal/i     then
-          extract_scale(field_type) == 0 ? :integer : :decimal
-        when /^char\(1\)$/i  then Firebird.emulate_booleans? ? :boolean : :string
-        when /^char/i        then :string
-        when /^blob\ssub_type\s(\d)/i
-          return :binary if $1 == '0'
-          return :text   if $1 == '1'
-        else
-          super
-        end
-      end
-
     end
 
     # @see ActiveRecord::ConnectionAdapters::JdbcAdapter#jdbc_column_class

data/lib/arjdbc/h2/adapter.rb

@@ -47,21 +47,6 @@ module ArJdbc
         limit
       end
 
-      def simplified_type(field_type)
-        case field_type
-        when /^bit|bool/i         then :boolean
-        when /^signed|year/i      then :integer
-        when /^real|double/i      then :float
-        when /^varchar/i          then :string
-        when /^longvarchar/i      then :text
-        when /^binary|raw|bytea/i then :binary
-        when /varbinary/i         then :binary # longvarbinary, varbinary
-        when /^blob|image|oid/i   then :binary
-        else
-          super
-        end
-      end
-
       # Post process default value from JDBC into a Rails-friendly format (columns{-internal})
       def default_value(value)
         # H2 auto-generated key default value

data/lib/arjdbc/hsqldb/adapter.rb

@@ -42,20 +42,6 @@ module ArJdbc
         limit
       end
 
-      def simplified_type(field_type)
-        case field_type
-        when /^nvarchar/i    then :string
-        when /^character/i   then :string
-        when /^longvarchar/i then :text
-        when /int/i          then :integer # TINYINT, SMALLINT, BIGINT, INT
-        when /real|double/i  then :float
-        when /^bit/i         then :boolean
-        when /binary/i       then :binary # VARBINARY, LONGVARBINARY
-        else
-          super
-        end
-      end
-
       # Post process default value from JDBC into a Rails-friendly format (columns{-internal})
       def default_value(value)
         # JDBC returns column default strings with actual single quotes around the value.

data/lib/arjdbc/informix/adapter.rb

@@ -28,11 +28,6 @@ module ArJdbc
       end
     end
 
-    # @see ActiveRecord::ConnectionAdapters::JdbcColumn#column_types
-    def self.column_selector
-      [ /informix/i, lambda { |cfg, column| column.extend(ColumnMethods) } ]
-    end
-
     JdbcConnection = ::ActiveRecord::ConnectionAdapters::InformixJdbcConnection
 
     # @see ActiveRecord::ConnectionAdapters::JdbcAdapter#jdbc_connection_class
@@ -45,20 +40,6 @@ module ArJdbc
       ::ActiveRecord::ConnectionAdapters::InformixColumn
     end
 
-    module ColumnMethods
-
-      private
-      # TODO: Test all Informix column types.
-      def simplified_type(field_type)
-        if field_type =~ /serial/i
-          :primary_key
-        else
-          super
-        end
-      end
-
-    end
-
     def modify_types(types)
       super(types)
       types[:primary_key] = "SERIAL PRIMARY KEY"
@@ -152,10 +133,6 @@ module ArJdbc
 end # module ::ArJdbc
 
 module ActiveRecord::ConnectionAdapters
-  class InformixColumn < JdbcColumn
-    include ::ArJdbc::Informix::ColumnMethods
-  end
-
   class InformixAdapter < JdbcAdapter
     include ::ArJdbc::Informix
   end

data/lib/arjdbc/jdbc/adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_record/version'
 require 'active_record/connection_adapters/abstract_adapter'
 
@@ -94,7 +96,7 @@ module ActiveRecord
         nil
       end
 
-      ADAPTER_NAME = 'JDBC'.freeze
+      ADAPTER_NAME = 'JDBC'
 
       # @return [String] the 'JDBC' adapter name.
       def adapter_name

data/lib/arjdbc/jdbc/adapter_require.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
 
   if defined? ConnectionAdapters::ConnectionSpecification::Resolver # 4.0
@@ -43,4 +45,4 @@ module ActiveRecord
 
   end
 
-end
+end

data/lib/arjdbc/jdbc/base_ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   class << Base
     m = Module.new do
@@ -12,4 +14,4 @@ module ActiveRecord
 
     self.prepend(m)
   end
-end
+end

data/lib/arjdbc/jdbc/callbacks.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord::ConnectionAdapters
   module Jdbc
     # ActiveRecord connection pool callbacks for JDBC.

data/lib/arjdbc/jdbc/column.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module Jdbc

data/lib/arjdbc/jdbc/connection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     # JDBC (connection) base class, custom adapters we support likely extend

data/lib/arjdbc/jdbc/connection_methods.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArJdbc
   ConnectionMethods = ::ActiveRecord::ConnectionHandling
 

data/lib/arjdbc/jdbc/error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   # Represents exceptions that have propagated up through the JDBC API.
   class JDBCError < WrappedDatabaseException

data/lib/arjdbc/jdbc/extension.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArJdbc
 
   # Defines an AR-JDBC extension. An extension consists of a declaration using

data/lib/arjdbc/jdbc/java.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'java'
 require 'arjdbc/jdbc/adapter_java'
 
@@ -10,4 +12,4 @@ module ActiveRecord
       Types = ::Java::JavaSql::Types
     end
   end
-end
+end

data/lib/arjdbc/jdbc/railtie.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 require 'arjdbc/railtie'
-ArJdbc.deprecate "require 'arjdbc/railtie' instead of 'arjdbc/jdbc/railtie'"
+ArJdbc.deprecate "require 'arjdbc/railtie' instead of 'arjdbc/jdbc/railtie'"

data/lib/arjdbc/jdbc/rake_tasks.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 load 'arjdbc/tasks.rb'
 
-ArJdbc.deprecate "load 'arjdbc/tasks.rb' instead of 'arjdbc/jdbc/rake_tasks.rb'"
+ArJdbc.deprecate "load 'arjdbc/tasks.rb' instead of 'arjdbc/jdbc/rake_tasks.rb'"

data/lib/arjdbc/jdbc/serialized_attributes_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'arjdbc/util/serialized_attributes'
 
-ArJdbc.deprecate "require 'arjdbc/util/serialized_attributes' instead of 'arjdbc/jdbc/serialized_attributes_helper'"
+ArJdbc.deprecate "require 'arjdbc/util/serialized_attributes' instead of 'arjdbc/jdbc/serialized_attributes_helper'"

data/lib/arjdbc/jdbc/type_cast.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_record/connection_adapters/column'
 
 module ActiveRecord::ConnectionAdapters

data/lib/arjdbc/jdbc/type_converter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     # I want to use JDBC's DatabaseMetaData#getTypeInfo to choose the best native types to

data/lib/arjdbc/mssql/adapter.rb

@@ -1,4 +1,4 @@
-# frozen_string_literal: false
+# frozen_string_literal: true
 
 ArJdbc.load_java_part :MSSQL
 
@@ -68,8 +68,19 @@ module ActiveRecord
         # configure_connection happens in super
         super(connection, logger, config)
 
-        unless mssql_major_version >= 11
-          raise "Your MSSQL #{mssql_version_year} is too old. This adapter supports MSSQL >= 2012."
+        if database_version < '11'
+          raise "Your #{mssql_product_name} #{mssql_version_year} is too old. This adapter supports #{mssql_product_name} >= 2012."
+        end
+      end
+
+      def self.database_exists?(config)
+        !!ActiveRecord::Base.sqlserver_connection(config)
+      rescue ActiveRecord::JDBCError => e
+        case e.message
+        when /Cannot open database .* requested by the login/
+          false
+        else
+          raise
         end
       end
 
@@ -101,6 +112,14 @@ module ActiveRecord
         true
       end
 
+      def supports_savepoints?
+        true
+      end
+
+      def supports_lazy_transactions?
+        true
+      end
+
       # The MSSQL datetime type doe have precision.
       def supports_datetime_with_precision?
         true
@@ -121,12 +140,23 @@ module ActiveRecord
         true
       end
 
+      def supports_insert_on_conflict?
+        false
+      end
+      alias supports_insert_on_duplicate_skip? supports_insert_on_conflict?
+      alias supports_insert_on_duplicate_update? supports_insert_on_conflict?
+      alias supports_insert_conflict_target? supports_insert_on_conflict?
+
+      def build_insert_sql(insert) # :nodoc:
+        # TODO: hope we can implement an upsert like feature
+        "INSERT #{insert.into} #{insert.values_list}"
+      end
+
       # Overrides abstract method which always returns false
       def valid_type?(type)
         !native_database_types[type].nil?
       end
 
-      # FIXME: to be reviewed.
       def clear_cache!
         reload_type_map
         super
@@ -170,22 +200,6 @@ module ActiveRecord
         result
       end
 
-      def arel_visitor # :nodoc:
-        ::Arel::Visitors::SQLServer.new(self)
-      end
-
-      def schema_creation # :nodoc:
-        MSSQL::SchemaCreation.new(self)
-      end
-
-      def create_table_definition(*args) # :nodoc:
-        MSSQL::TableDefinition.new(*args)
-      end
-
-      def update_table_definition(table_name, base) #:nodoc:
-        MSSQL::Table.new(table_name, base)
-      end
-
       # Returns the name of the current security context
       def current_user
         @current_user ||= select_value('SELECT CURRENT_USER')
@@ -208,15 +222,25 @@ module ActiveRecord
 
       alias_method :current_schema=, :default_schema=
 
-      # Overrides method in abstract adapter
-      # FIXME: This needs to be fixed the we find a way how to
-      # get the collation per column basis. At the moment we only use
-      # the global database collation
-      def case_sensitive_comparison(table, attribute, column, value)
-        if [:string, :text].include?(column.type) && collation && !collation.match(/_CS/)
-          table[attribute].eq(Arel::Nodes::Bin.new(value))
-        # elsif value.acts_like?(:string)
-        #   table[attribute].eq(Arel::Nodes::Bin.new(Arel::Nodes::BindParam.new))
+      # FIXME: This needs to be fixed when we implement the collation per
+      # column basis. At the moment we only use the global database collation
+      def default_uniqueness_comparison(attribute, value, klass) # :nodoc:
+        column = column_for_attribute(attribute)
+
+        if [:string, :text].include?(column.type) && collation && !collation.match(/_CS/) && !value.nil?
+          # NOTE: there is a deprecation warning here in the mysql adapter
+          # no sure if it's required.
+          attribute.eq(Arel::Nodes::Bin.new(value))
+        else
+          super
+        end
+      end
+
+      def case_sensitive_comparison(attribute, value)
+        column = column_for_attribute(attribute)
+
+        if [:string, :text].include?(column.type) && collation && !collation.match(/_CS/) && !value.nil?
+          attribute.eq(Arel::Nodes::Bin.new(value))
         else
           super
         end
@@ -248,6 +272,27 @@ module ActiveRecord
         MSSQL_VERSION_YEAR[mssql_major_version.to_i]
       end
 
+      def mssql_product_version
+        return @mssql_product_version if defined? @mssql_product_version
+
+        @mssql_product_version = @connection.database_product_version
+      end
+
+      def mssql_product_name
+        return @mssql_product_name if defined? @mssql_product_name
+
+        @mssql_product_name = @connection.database_product_name
+      end
+
+      def get_database_version # :nodoc:
+        MSSQLAdapter::Version.new(mssql_product_version)
+      end
+
+      def check_version # :nodoc:
+        # NOTE: hitting the database from here causes trouble when adapter
+        # uses JNDI or Data Source setup.
+      end
+
       def tables_with_referential_integrity
         schema_and_tables_sql = %(
           SELECT s.name, o.name
@@ -266,20 +311,22 @@ module ActiveRecord
 
       private
 
-      def translate_exception(exception, message)
+      def translate_exception(exception, message:, sql:, binds:)
         case message
         when /(cannot insert duplicate key .* with unique index) | (violation of unique key constraint)/i
-          RecordNotUnique.new(message)
+          RecordNotUnique.new(message, sql: sql, binds: binds)
         when /Lock request time out period exceeded/i
-          LockTimeout.new(message)
+          LockTimeout.new(message, sql: sql, binds: binds)
         when /The .* statement conflicted with the FOREIGN KEY constraint/
-          ActiveRecord::InvalidForeignKey.new(message)
+          InvalidForeignKey.new(message, sql: sql, binds: binds)
+        when /The .* statement conflicted with the REFERENCE constraint/
+          InvalidForeignKey.new(message, sql: sql, binds: binds)
         when /(String or binary data would be truncated)/i
-          ActiveRecord::ValueTooLong.new(message)
+          ValueTooLong.new(message, sql: sql, binds: binds)
         when /Cannot insert the value NULL into column .* does not allow nulls/
-          ActiveRecord::NotNullViolation.new(message)
+          NotNullViolation.new(message, sql: sql, binds: binds)
         when /Arithmetic overflow error converting expression/
-          ActiveRecord::RangeError.new(message)
+          RangeError.new(message, sql: sql, binds: binds)
         else
           super
         end
@@ -387,6 +434,28 @@ module ActiveRecord
         map.register_type 'ntext', MSSQL::Type::Ntext.new
         map.register_type 'image', MSSQL::Type::Image.new
       end
+
+      # Returns an array of Column objects for the table specified by +table_name+.
+      # See the concrete implementation for details on the expected parameter values.
+      # NOTE: This is ready, all implemented in the java part of adapter,
+      # it uses MSSQLColumn, SqlTypeMetadata, etc.
+      def column_definitions(table_name)
+       log('JDBC: GETCOLUMNS', 'SCHEMA') { @connection.columns(table_name) }
+      rescue => e
+        # raise translate_exception_class(e, nil)
+        # FIXME: this breaks one arjdbc test but fixes activerecord tests
+        # (table name alias). Also it behaves similarly to the CRuby adapter
+        # which returns an empty array too. (postgres throws a exception)
+        []
+      end
+
+      def arel_visitor # :nodoc:
+        ::Arel::Visitors::SQLServer.new(self)
+      end
+
+      def build_statement_pool
+        # NOTE: @statements is set in StatementCache module
+      end
     end
   end
 end

data/lib/arjdbc/mssql/column.rb

@@ -1,11 +1,15 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     # MSSQL specific extensions to column definitions in a table.
     class MSSQLColumn < Column
+      attr_reader :table_name
       def initialize(name, raw_default, sql_type_metadata = nil, null = true, table_name = nil, default_function = nil, collation = nil, comment: nil)
+        @table_name = table_name
         default = extract_default(raw_default)
 
-        super(name, default, sql_type_metadata, null, table_name, default_function, collation, comment: comment)
+        super(name, default, sql_type_metadata, null, default_function, collation: collation, comment: comment)
       end
 
       def extract_default(value)

data/lib/arjdbc/mssql/connection_methods.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ArJdbc::ConnectionMethods.module_eval do
 
   # Default connection method for MS-SQL adapter (`adapter: mssql`),
@@ -11,6 +13,8 @@ ArJdbc::ConnectionMethods.module_eval do
       return sqlserver_connection(config)
     end
 
+    config = config.deep_dup
+
     config[:adapter_spec] ||= ::ArJdbc::MSSQL
     config[:adapter_class] = ActiveRecord::ConnectionAdapters::MSSQLAdapter unless config.key?(:adapter_class)
 
@@ -31,7 +35,8 @@ ArJdbc::ConnectionMethods.module_eval do
     config[:connection_alive_sql] ||= 'SELECT 1'
 
     config[:url] ||= begin
-      url = "jdbc:jtds:sqlserver://#{config[:host]}:#{config[:port]}/#{config[:database]}"
+      url = ''.dup
+      url << "jdbc:jtds:sqlserver://#{config[:host]}:#{config[:port]}/#{config[:database]}"
       # Instance is often a preferrable alternative to port when dynamic ports are used.
       # If instance is specified then port is essentially ignored.
       url << ";instance=#{config[:instance]}" if config[:instance]
@@ -65,7 +70,8 @@ ArJdbc::ConnectionMethods.module_eval do
     config[:lock_timeout] ||= 5000
 
     config[:url] ||= begin
-      url = "jdbc:sqlserver://#{config[:host]}"
+      url = ''.dup
+      url << "jdbc:sqlserver://#{config[:host]}"
       url << ( config[:port] ? ":#{config[:port]};" : ';' )
       url << "databaseName=#{config[:database]};" if config[:database]
       url << "instanceName=#{config[:instance]};" if config[:instance]

data/lib/arjdbc/mssql/database_limits.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
  module ConnectionAdapters
    module MSSQL