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

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

[java] view plain copy

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

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

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

[java] view plain copy

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

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

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

[java] view plain copy

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

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

      Thread1                                                                                            Thread2

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

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

创建完毕，返回                                                                               

                                                                                                     此时进入registerConnectionPool

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

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

[java] view plain copy

1. import java.sql.*;  
2.   
3. public class TestForBug {  
4.     public static void main(String[] arug) {  
5.         new Thread(new T()).start();  
6.         new Thread(new T()).start();  
7.     }  
8. }  
9.   
10. class T implements Runnable {  
11.   
12.     public void run() {  
13.         Connection connection = null;  
14.         try {  
15.             Class.forName("org.logicalcobwebs.proxool.ProxoolDriver");  
16.             try {  
17.                 String url = "proxool.example:com.mysql.jdbc.Driver:jdbc:mysql://192.168.1.100:3306/mr";  
18.                 String username = "root";  
19.                 String password = "password";  
20.                 connection = DriverManager.getConnection(url, username, password);  
21.             } catch (SQLException e) {  
22.                 System.err.println("Problem getting connection" + e.toString());  
23.             }  
24.             if (connection != null) {  
25.                 System.out.println("Got connection :)");  
26.             } else {  
27.                 System.err.println("Didn't get connection, which probably means that no Driver accepted the URL");  
28.             }  
29.         } catch (ClassNotFoundException e) {  
30.             System.err.println("Couldn't find driver " + e.getMessage());  
31.         } finally {  
32.             try {  
33.                 if (connection != null) {  
34.                     connection.close();  
35.                 }  
36.             } catch (SQLException e) {  
37.                 System.err.println("Problem closing connection " + e);  
38.             }  
39.         }  
40.     }  
41. }  

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

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

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

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

查看下运行的堆栈，为：

测试代码

java.sql.DriverManager.

org.logicalcobwebs.proxool.Driver

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

结果真相大白了。

JDK5中：

[java] view plain copy

1. public static synchronized Connection getConnection(String url,   
2.     String user, String password) throws SQLException {    
3.         //省略  
4. ｝  

JDK6中：

[java] view plain copy

1. public static Connection getConnection(String url,   
2.     String user, String password) throws SQLException {    
3.         //省略  
4. }  

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

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

[java] view plain copy

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