proxool 0.9.1-解决 Attempt to register duplicate pool 异常

今天客户发来的日志中发现异常。该异常偶尔在程序启动的时候出现。

java.sql.SQLException: org.logicalcobwebs.proxool.ProxoolException: Attempt to register duplicate pool called 'pool'
	at org.logicalcobwebs.proxool.ProxoolDriver.connect(ProxoolDriver.java:109)
	at java.sql.DriverManager.getConnection(DriverManager.java:582)
	at java.sql.DriverManager.getConnection(DriverManager.java:185)

检查了下代码，似乎没有什么问题，一切都是按照proxool的教程中的example写的。于是把proxool的源代码下载，发现似乎真的有问题。

在类ProxoolDriver中，方法public Connection connect(String url, Properties info) 中，

if (!ConnectionPoolManager.getInstance().isPoolExists(alias)) {
     ProxoolFacade.registerConnectionPool(url, info, false);   
     cp = ConnectionPoolManager.getInstance().getConnectionPool(alias);
} 

这部分代码逻辑很简单，首先判断连接池的别名是否存在（isPoolExists)，如果不存在，创建；已经存在，则直接获得连接。

在第2行的registerConnectionPool中，再次判断，该别名是否存在，如果已经存在，则抛出异常：

if (!ConnectionPoolManager.getInstance().isPoolExists(alias)) {
     ConnectionPoolDefinition cpd = new ConnectionPoolDefinition(url, info, explicitRegister);
     registerConnectionPool(cpd);
} else {
     throw new ProxoolException("Attempt to register duplicate pool called '" + alias + "'");
}

那么，自然而然地考虑，这部分代码在并发的时候会出现问题。比如，下面两个thread, 分别调用getConnection

Thread1                                                                                                  Thread2

运行至(1), isPoolExists ? False                                                         运行至(1), isPoolExists ? False

运行至(2), 进入registerConnectionPool                                           运行至 (2) registerConnectionPool是static synchronized, 被阻塞住

创建完毕，返回                                                                                     此时进入registerConnectionPool

                                                                                                     If 处判断为已经存在(因为Thread1已经创建了），进入04行的else块，在05行抛出异常

然后写一小段程序来测试下：

import java.sql.*;

public class TestForBug {
    public static void main(String[] arug) {
        new Thread(new T()).start();
        new Thread(new T()).start();
    }
}

class T implements Runnable {

    public void run() {
        Connection connection = null;
        try {
            Class.forName("org.logicalcobwebs.proxool.ProxoolDriver");
            try {
                String url = "proxool.example:com.mysql.jdbc.Driver:jdbc:mysql://192.168.1.100:3306/mr";
                String username = "root";
                String password = "password";
                connection = DriverManager.getConnection(url, username, password);
            } catch (SQLException e) {
                System.err.println("Problem getting connection" + e.toString());
            }
            if (connection != null) {
                System.out.println("Got connection :)");
            } else {
                System.err.println("Didn't get connection, which probably means that no Driver accepted the URL");
            }
        } catch (ClassNotFoundException e) {
            System.err.println("Couldn't find driver " + e.getMessage());
        } finally {
            try {
                if (connection != null) {
                    connection.close();
                }
            } catch (SQLException e) {
                System.err.println("Problem closing connection " + e);
            }
        }
    }
}

这段程序十有八九会抛出那个异常。

这就怪了。Proxool用的也很多，为啥会犯这么低级的错误呢。后来在跟踪的时候，发现：

如果运行在JDK5下，就没问题；

如果运行在JDK6下，则有问题。

查看下运行的堆栈，为：

测试代码

java.sql.DriverManager.

org.logicalcobwebs.proxool.Driver

既然测试代码和proxool.Driver都没改变，那么就看看java.sql.DriverManager是怎么回事。

结果真相大白了。

JDK5中：

public static synchronized Connection getConnection(String url, 
	String user, String password) throws SQLException {  
        //省略
｝

JDK6中：

public static Connection getConnection(String url, 
	String user, String password) throws SQLException {  
        //省略
}

也就是，上面说的并发的问题，在JDK5中是同步的。因为DriverManager里面的方法就是同步的。但是在6中就不是了。这就是为啥上面的isPoolExists的代码失效的原因。也就是说，在JDK6中，proxool可能会出现这个问题。

那么解决也很简单，修改ProxoolDriver，把connect方法加上synchronized即可：

public synchronized Connection connect(String url, Properties info) {
    //省略
}