]> source.dussan.org Git - sonarqube.git/commitdiff
Remove unused Ruby code : JNDI support in activerecord adapters
authorSimon Brandhof <simon.brandhof@gmail.com>
Mon, 16 Jan 2012 16:11:08 +0000 (17:11 +0100)
committerSimon Brandhof <simon.brandhof@gmail.com>
Mon, 16 Jan 2012 16:11:08 +0000 (17:11 +0100)
sonar-server/src/main/webapp/WEB-INF/vendor/gems/activerecord-jdbc-adapter-1.1.3/lib/arjdbc/jdbc/connection.rb
sonar-server/src/main/webapp/WEB-INF/vendor/gems/activerecord-oracle_enhanced-adapter-1.4.0/lib/active_record/connection_adapters/oracle_enhanced_jdbc_connection.rb

index fec934983a682d451997df79e07aa5daa80da5db..0d37eace4fefa69485c3f6dbd209aa2946858fe3 100644 (file)
@@ -13,34 +13,11 @@ module ActiveRecord
           config[:connection_alive_sql] ||= "select 1"
 
           # sonar
-          # ActiveRecord must transfer the responsibility of connection pool to the Sonar,
-          # even if JNDI datasource is not used.
+          # The connection pool is managed by Sonar (commons-dbcp) but not by ActiveRecord
           @jndi_connection = true
-          # /sonar
           @connection = nil
-          if config[:jndi]
-            begin
-              configure_jndi
-            rescue => e
-              warn "JNDI data source unavailable: #{e.message}; trying straight JDBC"
-              configure_jdbc
-            end
-          else
-            configure_jdbc
-          end
-        end
-
-        def configure_jndi
-          jndi = config[:jndi].to_s
-          ctx = javax.naming.InitialContext.new
-          ds = ctx.lookup(jndi)
-          @connection_factory = JdbcConnectionFactory.impl do
-            ds.connection
-          end
-          unless config[:driver]
-            config[:driver] = connection.meta_data.connection.java_class.name
-          end
-          @jndi_connection = true
+          configure_jdbc
+          # /sonar
         end
 
         def configure_url
index 6a6694c1e2de91bf711202e14dac12063ec89e0e..7f709da824f4f6a502bdc286291b20639d71e7c0 100644 (file)
@@ -56,17 +56,15 @@ module ActiveRecord
       end
 
       #sonar
-      # modified method to support JNDI connections
+      # Commons DBCP is used to manage pool of JDBC connections.
       def new_connection(config)
         @raw_connection = ::Java::OrgSonarServerUi::JRubyFacade.getInstance().getConnection()
+
         if @raw_connection.respond_to?(:getInnermostDelegate)
           @pooled_connection = @raw_connection
-          @raw_connection = @raw_connection.innermost_delegate
-        elsif @raw_connection.respond_to?(:getUnderlyingConnection)
-          @pooled_connection = @raw_connection
-          @raw_connection = @raw_connection.underlying_connection
+          @raw_connection = @raw_connection.getInnermostDelegate()
         else
-          raise ArgumentError, "JDBC Datasource not supported. Please use Commons DBCP or Tomcat Connection Pool"
+          raise ArgumentError, "This JDBC connection is not managed by Commons DBCP"
         end
         username = @raw_connection.meta_data.user_name