请不要吞没和忽略 java.lang.InterruptedException

[chaosky]

你好:

DruidDataSource.java 的 1281 的 lock.lockInterruptibly() 行会产生 InterruptedException, 请不要在 catch (Throwable e) 里面忽略异常, 这样上层无法收到中断. 可以重新抛出异常或者重新设置中断状态. 抛出异常对客户来说比较有好, 可以和jdk库抛出的中断统一处理. 也请检查其他你们库中的其他地方是否忽略了中断. 谢谢.

关联代码为:

            final long lastActiveTimeMillis = System.currentTimeMillis();
            lock.lockInterruptibly();  // 这里产生的中断异常请重新抛出
            try {
                activeCount--;
                closeCount++;

                putLast(holder, lastActiveTimeMillis);
                recycleCount++;
            } finally {
                lock.unlock();
            }
        } catch (Throwable e) {
            holder.clearStatementCache();

            if (!holder.isDiscard()) {
                this.discardConnection(physicalConnection);
                holder.setDiscard(true);
            }

            LOG.error("recyle error", e);
            recycleErrorCount.incrementAndGet();
        }

[wenshao]

不吃掉也抛不出来啊

[chaosky]

那就请重新设置中断状态(也可以异步抛出的). 直接忽略的话上层通过中断来通知业务停止处理时, 如果调用了druid的相关方法会导致业务层收不到中断通知.

[xiaxianggo]

一般在什么情况下这边会产生 InterruptedException，其他数据库没怎么遇到 最近用odbc连接azure，这边频繁中断

[darren-fu]

对待issues的态度是真差