activerecord-jdbc-adapter 51.7-java → 52.0-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a472f8ed7911f754bafc83ddf5d34fa061fda0d8b78a0a0d263e54ac4ae59ffa
-  data.tar.gz: 1fd596cb2615ab284b647d6546e40eab484c9718890d0ee90e5a9220610762cc
+  metadata.gz: d4e30effdf5519398605801e776e032f385bba8da0008c15bdda99426ac0b2ee
+  data.tar.gz: 5c3075c02b57070186442fd4d8e88252ffab2db0f06ebed390dcdeb7556a27ba
 SHA512:
-  metadata.gz: 7686dceb4704955a36053011ae7da81abc6384bd8ad213a171f99c2becb1d814138192f6f05923e2bfd1434e395b59e38418ffcd57ab7324d352c6a9a96b4bf2
-  data.tar.gz: 0030c4eb8f4e61615f7d30b4a1693fa26729144e32484bfc02edccf418fe339f79736c0c4039e89a6e81159d9e45dbbf10cf18deab6bd2f5c3ee75ffa24db137
+  metadata.gz: 01c8df592aec36d47474c1968ff73e39f68414a3a3fbde27fcaa5c24fe078ee8cddb2b3f750d580660d567fb3323511e7ad9781913bb6fe658797a9445b42bce
+  data.tar.gz: 1940e41e96e0ac3af73bb0bbdd1dcc75c9c07554e2d3c05a8cc3bf3562cbd9fa9593919a8159c151577463ad676be42b92b2b61089efdebc9ad94fff34596ec7

data/.gitignore

@@ -2,7 +2,6 @@
 *.tgz
 *~
 *.log
-/log/
 patches*
 *#
 TAGS
@@ -31,4 +30,4 @@ Gemfile.lock
 .idea
 .settings
 activerecord-jdbc.iml
-lib/arjdbc/jdbc/adapter_java.jar
+lib/arjdbc/jdbc/adapter_java.jar

data/.travis.yml

@@ -1,27 +1,9 @@
+language: ruby
 sudo: false
-dist: xenial
-
-services:
-  - mysql
-addons:
-  postgresql: 9.4
-
+bundler_args: --without development
+script: bundle exec rake ${TEST_PREFIX}test_$DB
 before_install:
   - unset _JAVA_OPTIONS
-  - rvm @default,@global do gem uninstall bundler -a -x -I || true
-  - gem install bundler -v "~>1.17.3"
-install:
-- bundle install --retry 3 --without development
-# to fix this issue: https://travis-ci.community/t/mariadb-10-1-fails-to-install-on-xenial/3151/3
-- mysql -u root -e 'CREATE USER IF NOT EXISTS travis@localhost; GRANT ALL ON *.* TO travis@localhost;' || true
-
-language: ruby
-rvm:
-  - jruby-9.1.16.0
-jdk:
-  - openjdk8
-
-script: bundle exec rake ${TEST_PREFIX}test_$DB
 before_script:
   - echo "JAVA_OPTS=$JAVA_OPTS"
   - export JRUBY_OPTS="-J-Xms64M -J-Xmx1024M"
@@ -39,15 +21,18 @@ before_script:
     mysql -e "grant all privileges on activerecord_unittest.* to rails@localhost;" && \
     mysql -e "grant all privileges on activerecord_unittest2.* to rails@localhost;" && \
     mysql -e "grant all privileges on inexistent_activerecord_unittest.* to rails@localhost;" && \
-    mysql -e "CREATE DATABASE activerecord_unittest DEFAULT CHARACTER SET utf8 COLLATE utf8_general_ci;" && \
-    mysql -e "CREATE DATABASE activerecord_unittest2 DEFAULT CHARACTER SET utf8 COLLATE utf8_general_ci;" \
+    mysql -e "CREATE DATABASE activerecord_unittest;" && \
+    mysql -e "CREATE DATABASE activerecord_unittest2;" \
     || true
   - |
     [ "$DB" == "postgresql" ] && [ "$TEST_PREFIX" == "rails:" ] && \
     psql  -c "create database activerecord_unittest;" -U postgres && \
     psql  -c "create database activerecord_unittest2;" -U postgres \
     || true
-
+rvm:
+  - jruby-9.1.16.0
+jdk:
+  - openjdk8
 env:
   - DB=mysql2 PREPARED_STATEMENTS=false
   - DB=mysql2 PREPARED_STATEMENTS=true
@@ -58,29 +43,19 @@ env:
   - DB=postgresql PREPARED_STATEMENTS=true INSERT_RETURNING=true
   - DB=sqlite3 PREPARED_STATEMENTS=false
   - DB=sqlite3 PREPARED_STATEMENTS=true
-  - DB=jndi PREPARED_STATEMENTS=false
-  - DB=jndi PREPARED_STATEMENTS=true
+  #- DB=jndi PREPARED_STATEMENTS=false
+  #- DB=jndi PREPARED_STATEMENTS=true
+branches:
+  only:
+    - master
+    - /.*-stable$/
+    - /^test-.*/
+    - /maintenance|support/
+    - /.*dev$/
 matrix:
   allow_failures:
     - rvm: jruby-head
   include:
-    # jruby-9.2
-    - rvm: jruby-9.2.6.0
-      env: DB=mysql2
-    - rvm: jruby-9.2.6.0
-      env: DB=postgresql
-    - rvm: jruby-9.2.6.0
-      env: DB=sqlite3
-    # jruby-9.2 + Java 11
-    - rvm: jruby-9.2.6.0
-      env: DB=mysql2
-      jdk: openjdk11
-    - rvm: jruby-9.2.6.0
-      env: DB=postgresql
-      jdk: openjdk11
-    - rvm: jruby-9.2.6.0
-      env: DB=sqlite3
-      jdk: openjdk11
     # jruby-head
     - rvm: jruby-head
       env: DB=mysql2
@@ -90,15 +65,15 @@ matrix:
       env: DB=sqlite3
     # testing against MariaDB
     - addons:
-        mariadb: 10.2
+        mariadb: '10.0'
       env: DB=mariadb PREPARED_STATEMENTS=false
     - addons:
-        mariadb: 10.3
+        mariadb: '10.1'
       env: DB=mariadb PREPARED_STATEMENTS=true
     # Rails test-suite :
-    - env: DB=mysql2     TEST_PREFIX="rails:" AR_VERSION="v5.1.7" # PS off by default
-    - env: DB=mysql2     TEST_PREFIX="rails:" AR_VERSION="v5.1.7" PREPARED_STATEMENTS=true
-    - env: DB=mysql2     TEST_PREFIX="rails:" AR_VERSION="v5.1.7" DRIVER=MariaDB
-    - env: DB=postgresql TEST_PREFIX="rails:" AR_VERSION="v5.1.7" # PS on by default
-    - env: DB=postgresql TEST_PREFIX="rails:" AR_VERSION="v5.1.7" PREPARED_STATEMENTS=false
-    - env: DB=sqlite3    TEST_PREFIX="rails:" AR_VERSION="v5.1.7"
+    - env: DB=mysql2     TEST_PREFIX="rails:" AR_VERSION="v5.2.0" # PS off by default
+    - env: DB=mysql2     TEST_PREFIX="rails:" AR_VERSION="v5.2.0" PREPARED_STATEMENTS=true
+    - env: DB=mysql2     TEST_PREFIX="rails:" AR_VERSION="v5.2.0" DRIVER=MariaDB
+    - env: DB=postgresql TEST_PREFIX="rails:" AR_VERSION="v5.2.0" # PS on by default
+    - env: DB=postgresql TEST_PREFIX="rails:" AR_VERSION="v5.2.0" PREPARED_STATEMENTS=false
+    - env: DB=sqlite3    TEST_PREFIX="rails:" AR_VERSION="v5.2.0"

data/README.md

@@ -5,16 +5,15 @@
 ActiveRecord-JDBC-Adapter (AR-JDBC) is the main database adapter for Rails'
 *ActiveRecord* component that can be used with [JRuby][0].
 ActiveRecord-JDBC-Adapter provides full or nearly full support for:
-**MySQL**, **PostgreSQL**, **SQLite3** and **MSSQL*** (SQLServer).
+**MySQL**, **PostgreSQL**, **SQLite3**.  In the near future there are plans to
+add support **MSSQL**. Unless we get more contributions we will not be going
+beyond these four adapters.  Note that the amount of work needed to get
+another adapter is not huge but the amount of testing required to make sure
+that adapter continues to work is not something we can do with the resources
+we currently have.
 
-Unless we get more contributions we will not be supporting more adapters.
-Note that the amount of work needed to get another adapter is not huge but
-the amount of testing required to make sure that adapter continues to work
-is not something we can do with the resources we currently have.
-
-- for **Oracle** database users you are encouraged to use
-  https://github.com/rsim/oracle-enhanced
-- **MSSQL** adapter's gem parts reside in a [separate repository][8]
+For Oracle database users you are encouraged to use
+https://github.com/rsim/oracle-enhanced.
 
 Version **50.x** supports Rails version 5.0.x and it lives on branch 50-stable.
 Version **51.x** supports Rails version 5.1.x and is currently on master until
@@ -33,7 +32,6 @@ adapters are available:
   - MySQL (`activerecord-jdbcmysql-adapter`)
   - PostgreSQL (`activerecord-jdbcpostgresql-adapter`)
   - SQLite3 (`activerecord-jdbcsqlite3-adapter`)
-  - MSSQL (`activerecord-jdbcsqlserver-adapter`)
 
 2. If you're generating a new Rails application, use the following command:
 
@@ -168,4 +166,4 @@ license the database's drivers are licensed. See each driver gem's LICENSE.txt.
 [5]: https://github.com/jruby/activerecord-jdbc-adapter/wiki
 [6]: https://webchat.freenode.net/?channels=#jruby
 [7]: http://badge.fury.io/rb/activerecord-jdbc-adapter
-[8]: https://github.com/jruby/activerecord-jdbcsqlserver-adapter
+[8]: https://github.com/jruby/activerecord-jdbc-adapter/wiki/Migrating-from-1.2.x-to-1.3.0

data/Rakefile

@@ -1,29 +1,3 @@
-# Common usage
-#
-#   rake build:adapters - to build all specific adapter gems and the base gem
-#   rake release:do - build:adapters + git tag + push gems
-#
-# Environment variables used by this Rakefile:
-#
-# INCLUDE_JAR_IN_GEM [default task - false, other taks - true]:
-#   Note: This is something you should not normally have to set.
-#   For local development we always will end up including the jar file
-#   in any task which generates our main gem.  The wrinkle to this
-#   is when we do a custom github link in bundler:
-#
-#      gem 'ar-jdbc', github: '...'
-#
-#   Because we stopped committing a .jar file for every change and so when
-#   we  include a gem like this it clones the repository and does a default
-#   build in rake.  This in turn will end up forcing a compile to generate
-#   that jar (similar to how c-extensions compile at the time of install).
-#   For shipped gems we do include the jar so that people do not require
-#   this compile step.
-#
-# NOOP [release:do - false]
-#
-#   No commands or gem pushing during a release.
-
 require 'rake/clean'
 
 CLEAN.include 'derby*', 'test.db.*', '*test.sqlite3', 'test/reports'
@@ -64,8 +38,7 @@ rake = lambda { |task| ruby "-S", "rake", task }
 
 desc "Build #{gem_name} gem into the pkg directory."
 task :build => :jar do
-  include_jar = ENV['INCLUDE_JAR_IN_GEM'] || 'true'
-  sh("INCLUDE_JAR_IN_GEM=#{include_jar} gem build -V '#{gemspec_path}'") do
+  sh("RELEASE=#{ENV['RELEASE']} gem build -V '#{gemspec_path}'") do
     gem_path = built_gem_path.call
     file_name = File.basename(gem_path)
     FileUtils.mkdir_p(File.join(base_dir, 'pkg'))
@@ -85,6 +58,7 @@ end
 
 desc "Releasing AR-JDBC gems (use NOOP=true to disable gem pushing)"
 task 'release:do' do
+  ENV['RELEASE'] = 'true' # so that .gemspec is built with adapter_java.jar
   Rake::Task['build'].invoke
   Rake::Task['build:adapters'].invoke
 
@@ -262,16 +236,19 @@ if defined? JRUBY_VERSION
     end
 
     classpath = []
-    [ 'java.class.path', 'sun.boot.class.path' ].map { |key| ENV_JAVA[key] }.each do |v|
-      classpath += v.split(File::PATH_SEPARATOR).find_all { |jar| jar =~ /jruby/i } if v
+    ENV_PATHS = [ 'java.class.path', 'sun.boot.class.path' ]
+    ENV_PATHS.map { |key| ENV_JAVA[key] }.compact.each do |v|
+      classpath += v.split(File::PATH_SEPARATOR).find_all { |jar| jar =~ /jruby/i }
     end
-    # Using Java 9+.  Let's try and infer jruby.jar location from rbconfig
-    if classpath.empty?; require 'rbconfig'
-      libdir = RbConfig::CONFIG['libdir']
-      if libdir.start_with? 'classpath:'
+
+    # Using Java 9+.  Let's try and infer jruby jar location from rbconfig
+    if classpath.empty?
+      require 'rbconfig'
+      libdir = RbConfig::CONFIG["libdir"]
+      if libdir.start_with? "classpath:"
         error "Cannot build activerecord-jdbc with jruby-complete"
       end
-      classpath << File.join(libdir, 'jruby.jar')
+      classpath << File.join(libdir, "jruby.jar")
     end
     
     classpath += driver_jars
@@ -279,26 +256,11 @@ if defined? JRUBY_VERSION
 
     source_files = FileList[ 'src/java/**/*.java' ]
 
-    version = lambda do
-      begin
-        require 'arjdbc/version'
-      rescue LoadError
-        path = File.expand_path('lib', File.dirname(__FILE__))
-        unless $LOAD_PATH.include?(path)
-          $LOAD_PATH << path; retry
-        end
-      end
-
-      gem_version = Gem::Version.create(ArJdbc::VERSION)
-      if gem_version.segments.last == 'DEV'
-        gem_version.segments[0...-1] # 50.0.DEV -> 50.0
-      else
-        gem_version.segments.dup
-      end
-    end
+    require 'tmpdir'
 
-    require 'tmpdir'; Dir.mktmpdir do |classes_dir|
-      # Cross-platform way of finding an executable in the $PATH. Thanks to @mislav
+    Dir.mktmpdir do |classes_dir|
+      # Cross-platform way of finding an executable in the $PATH.
+      # Thanks to @mislav
       which = lambda do |cmd|
         exts = ENV['PATHEXT'] ? ENV['PATHEXT'].split(';') : ['']
         ENV['PATH'].split(File::PATH_SEPARATOR).map do |path|
@@ -322,24 +284,7 @@ if defined? JRUBY_VERSION
       # avoid environment variable expansion using backslash
       # class_files.gsub!('$', '\$') unless windows?
       # args = class_files.map { |path| [ "-C #{classes_dir}", path ] }.flatten
-
-      if ENV['INCLUDE_JAR_IN_GEM'] == 'true'; require 'tempfile'
-        manifest  = "Built-Time: #{Time.now.utc.strftime('%Y-%m-%d %H:%M:%S')}\n"
-        manifest += "Built-JRuby: #{JRUBY_VERSION}\n"
-        manifest += "Specification-Title: ActiveRecord-JDBC\n"
-        manifest += "Specification-Vendor: JRuby\n"
-        manifest += "Specification-Version: #{version.call[0].to_s.split('').join('.')}\n" # AR VERSION (52 -> 5.2)
-        manifest += "Implementation-Vendor: The JRuby Team\n"
-        manifest += "Implementation-Version: #{version.call.join('.')}\n"
-        manifest  = Tempfile.new('MANIFEST').tap { |f| f << manifest; f.close }.path
-      end
-
-      args = []; opts = '-cf'
-      if manifest
-        opts = "#{opts}m"
-        args = [ "#{manifest}" ]
-      end
-      args += [ '-C', "#{classes_dir}/ ." ]
+      args = [ '-C', "#{classes_dir}/ ." ] # args = class_files
 
       jar_path = jar_file
       if ext_lib_dir = ENV['RUBYLIBDIR']
@@ -349,7 +294,7 @@ if defined? JRUBY_VERSION
       unless jar = which.call('jar')
         warn "could not find jar tool, please make sure it's on the PATH"
       end
-      sh("#{jar} #{opts} #{jar_path} #{args.join(' ')}") do |ok|
+      sh("#{jar} cf #{jar_path} #{args.join(' ')}") do |ok|
         raise 'could not build .jar extension - packaging failure' unless ok
       end
       cp jar_path, jar_file if ext_lib_dir # NOTE: hopefully RG won't mind?!
@@ -357,6 +302,6 @@ if defined? JRUBY_VERSION
   end
 else
   task :jar do
-    warn "please run `rake jar' under JRuby to re-compile the native (Java) extension"
+    puts "please run `rake jar' under JRuby to re-compile the native (Java) extension"
   end
 end

data/activerecord-jdbc-adapter.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |gem|
     reject { |f| f =~ /^(gemfiles)/ } # no tests - no Gemfile_s appraised ...
   gem.files += ['lib/arjdbc/jdbc/adapter_java.jar'] #if ENV['RELEASE'].eql?('true')
 
-  if ENV['INCLUDE_JAR_IN_GEM'] != 'true' # @see Rakefile
+  if ENV['RELEASE'] != 'true' # @see Rakefile
     gem.extensions << 'Rakefile' # to support auto-building .jar with :git paths
 
     #gem.add_runtime_dependency 'jar-dependencies', '~> 0.1' # development not enough!
@@ -44,7 +44,7 @@ Gem::Specification.new do |gem|
   gem.executables = gem.files.grep(%r{^bin/}).map { |f| File.basename(f) }
   gem.test_files = gem.files.grep(%r{^test/})
 
-  gem.add_dependency 'activerecord', '~> 5.1.0'
+  gem.add_dependency 'activerecord', '~> 5.2.0'
 
   #gem.add_development_dependency 'test-unit', '2.5.4'
   #gem.add_development_dependency 'test-unit-context', '>= 0.3.0'

data/lib/active_record/connection_adapters/mssql_adapter.rb

@@ -0,0 +1 @@
+require 'arjdbc/mssql'

data/lib/active_record/connection_adapters/sqlserver_adapter.rb

@@ -0,0 +1 @@
+require 'arjdbc/mssql'

data/lib/arjdbc/abstract/core.rb

@@ -22,7 +22,7 @@ module ArJdbc
 
         connection.configure_connection # will call us (maybe)
       end
-
+      
       # Retrieve the raw `java.sql.Connection` object.
       # The unwrap parameter is useful if an attempt to unwrap a pooled (JNDI)
       # connection should be made - to really return the 'native' JDBC object.
@@ -56,21 +56,11 @@ module ArJdbc
         case e
           when SystemExit, SignalException, NoMemoryError then e
           when ActiveModel::RangeError, TypeError, RuntimeError then e
-          else super
+          else ActiveRecord::StatementInvalid.new(message)
         end
       end
 
-      def extract_raw_bind_values(binds)
-        binds.map(&:value_for_database)
-      end
 
-      # this version of log() automatically fills type_casted_binds from binds if necessary
-      def log(sql, name = "SQL", binds = [], type_casted_binds = [], statement_name = nil)
-        if binds.any? && (type_casted_binds.nil? || type_casted_binds.empty?)
-          type_casted_binds = ->{ extract_raw_bind_values(binds) }
-        end
-        super
-      end
     end
   end
 end

data/lib/arjdbc/abstract/database_statements.rb

@@ -10,13 +10,11 @@ module ArJdbc
       NO_BINDS = [].freeze
 
       def exec_insert(sql, name = nil, binds = NO_BINDS, pk = nil, sequence_name = nil)
-        binds = convert_legacy_binds_to_attributes(binds) if binds.first.is_a?(Array)
-
         if without_prepared_statement?(binds)
-          log(sql, name) { @connection.execute_insert_pk(sql, pk) }
+          log(sql, name) { @connection.execute_insert(sql) }
         else
           log(sql, name, binds) do
-            @connection.execute_insert_pk(sql, binds, pk)
+            @connection.execute_insert(sql, binds)
           end
         end
       end
@@ -24,22 +22,18 @@ module ArJdbc
       # It appears that at this point (AR 5.0) "prepare" should only ever be true
       # if prepared statements are enabled
       def exec_query(sql, name = nil, binds = NO_BINDS, prepare: false)
-        binds = convert_legacy_binds_to_attributes(binds) if binds.first.is_a?(Array)
-
         if without_prepared_statement?(binds)
           log(sql, name) { @connection.execute_query(sql) }
         else
           log(sql, name, binds) do
-            # this is different from normal AR that always caches
-            cached_statement = fetch_cached_statement(sql) if prepare && @jdbc_statement_cache_enabled
+            # It seems that #supports_statement_cache? is defined but isn't checked before setting "prepare" (AR 5.0)
+            cached_statement = fetch_cached_statement(sql) if prepare && supports_statement_cache?
             @connection.execute_prepared_query(sql, binds, cached_statement)
           end
         end
       end
 
       def exec_update(sql, name = nil, binds = NO_BINDS)
-        binds = convert_legacy_binds_to_attributes(binds) if binds.first.is_a?(Array)
-
         if without_prepared_statement?(binds)
           log(sql, name) { @connection.execute_update(sql) }
         else
@@ -48,10 +42,30 @@ module ArJdbc
       end
       alias :exec_delete :exec_update
 
+      # overridden to support legacy binds
+      def insert(arel, name = nil, pk = nil, id_value = nil, sequence_name = nil, binds = [])
+        binds = convert_legacy_binds_to_attributes(binds) if binds.first.is_a?(Array)
+        super
+      end
+      alias create insert
+
+      # overridden to support legacy binds
+      def update(arel, name = nil, binds = [])
+        binds = convert_legacy_binds_to_attributes(binds) if binds.first.is_a?(Array)
+        super
+      end
+
+      # overridden to support legacy binds
+      def delete(arel, name = nil, binds = [])
+        binds = convert_legacy_binds_to_attributes(binds) if binds.first.is_a?(Array)
+        super
+      end
+
       def execute(sql, name = nil)
         log(sql, name) { @connection.execute(sql) }
       end
 
+      # overridden to support legacy binds
       def select_all(arel, name = nil, binds = NO_BINDS, preparable: nil)
         binds = convert_legacy_binds_to_attributes(binds) if binds.first.is_a?(Array)
         super