做项目的时候，遇见空异常，而且不是经常的，本来想将就的放过，可考虑到偶尔影响用户的正常使用，对用户体验非常不好，还是要花些时间查找问题的根源。结果如预料的那样，跟转发来的这篇博文讲述的性质一模一样。
同时再赞叹一声，转发来的这位博主，写的很详尽。下面是原文：
今天客户发来的日志中发现异常。该异常偶尔在程序启动的时候出现。
- 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即可：
已有 0人发表留言，猛击->> 这里<<-参与讨论
ITeye推荐
