多线程问题导致的JDBMonitor的bug分析
来源:互联网 发布:java图书馆借阅系统 编辑:程序博客网 时间:2024/05/22 08:49
以前我一直是用使用数据源的系统测试JDBMonitor,昨天我准备把JDBMonitor嵌入到一个小的jsp留言板中,这个留言板写的非常简单,数据库操作也是connection随取随用、用完了就关这种最简单的形式,这倒是帮助我发现了一个超级大bug。
在记事本运行中常常不规律的DataBaseDBListener的logSQL方法报空指针错误,是在logStatement.setString(1, randomGUID.toString());这一句抛出的,我跟踪到mysql驱动的内部(我使用的是mysql数据库做DataBaseDBListener的输出数据库),发现在setString的实现中会去调用connection的方法,而此时connection已经是null了,所以会报空指针错误。
从其不规律的特点我判断出应该是多线程导致的问题。我跟踪发现,有的时候是先调用DataBaseDBListener的close后logSql才被调用。经过分析找出了问题所在,问题就在DBLogger中内部类LogConsumer的startConsumer方法,这个是修改以前的代码:
for (;;)
{
SQLInfo info = (SQLInfo) channel.take();
if(info==null)
{
ontinue;
}
for (int i = 0, n = dbListeners.length; i < n; i++)
{
dbListeners[i].logSql(info);
}
}
在调用channel.take()以后,这个SQLInfo就从channel中去除了。这时如果DBLogger的方法被调用,那么即使dbListeners[i].logSql(info)仍然在运行,但是DBLogger会认为channel已经空了,所有的dbListeners都可以被close了,而在DataBaseDBListener的close方法中会把输出目标数据库connection关掉,那么此时如果DataBaseDBListener的logSql方法还在调用的话,那么一旦使用这个connection,那么就会出现错误了。
我是如下修改的:
修改BlockedChannel类,去掉其take方法,增加一个peek方法和一个remove方法,peek方法是从channel中查出一个对象,但是不把它从channel中移走,只有调用remove方法后才会被移走。
修改DBLogger中内部类LogConsumer的startConsumer方法为如下:
for (;;)
{
SQLInfo info = (SQLInfo) channel.peek();
if(info==null)
{
continue;
}
for (int i = 0, n = dbListeners.length; i < n; i++)
{
dbListeners[i].logSql(info);
}
channel.remove(info);
}
也就是只有SQLInfo被处理完了,才会从channel中移走。
看来JDBMonitor还是要多多测试多多验证,我近期准备对JDBMonitor进行性能测试,看看有没有并发问题有没有性能问题,不要再急着加新功能了,把这些基础的功能做好再说,否则别人用着用着就出现中断性错误了,那将会是比缺少功能更可怕的事情。一位同事曾经说过一句话:“做的越多错的越多。不要你做的多好,只要把已经做完的东西做的更好就可以了。”,看来确实有一定道理呀。把基本的东西做好再说,否则被人骂不专业就惨了。
- 多线程问题导致的JDBMonitor的bug分析
- jdbmonitor的配置
- 经验问题导致的bug
- Log4j和JDBMonitor的比较
- SimpleDateFormat导致的多线程问题
- 数据库相关:Log4j和JDBMonitor的比较
- 多线程导致的iOS闪退分析
- if导致的bug
- snprintf 导致的bug
- 粗心导致的bug
- munmap导致的BUG
- 一个时序混乱导致的bug分析过程记录
- jdk1.6.0_29的bug导致c3p0获取sqlserver连接时被hang住甚至deadlock问题的分析
- Android权限问题导致的系统调用bug
- 工作流jbpm3.1.2 bug 导致数据库连接池满的问题
- Bug:StampedLock的中断问题导致CPU爆满
- 设计模式--JDBMonitor的基本原理探究
- 锁表导致的BUG
- 高级:java学习彻底明白Java的IO系统
- Java专业术语标准化规范
- ROW_NUMBER() OVER ,ROWNUM , OVER()
- 学会如何沟通?
- shel编程中如何避免死锁。(原子操作概念)
- 多线程问题导致的JDBMonitor的bug分析
- 在js中的提交 带有参数连接的那种
- windows mobile中利用WMPLib播放MP3文件(转载)
- 字符集本地化(locale)与输入法系列讲座-----(3) truetype造字程序详解
- Java入门--生成可执行jar文件的教程
- 一些有用的正则表达式收集
- BUG分类标准
- 博客开了
- 分析Android 根文件系统启动过程(init守护进程分析)