日常Bug排查-抛异常不回滚

前言

日常Bug排查系列都是一些简单Bug排查,笔者将在这里介绍一些排查Bug的简单技巧,同时顺便积累素材。

Bug现场

最近有人反映java应用操作数据库的时候,抛异常不回滚。这还了得,不过笔者笃定肯定是用法的锅,不然就全乱套了。所以笔者去Review他的代码。

代码片段

@Transacion(value="x")
public void s1() throw MyException{
	update(1);
	throwBusinessException();
 	update(2);
}

乍看上去没啥问题。
日常Bug排查-抛异常不回滚-LMLPHP

思路

笔者用@Transaction注解也用了好几年了,从来没遇到过抛异常不回滚的情况。看他的用法也和笔者差不多呀?
然后笔者稍微思索了会,发现我写的代码和出问题的这一段稍稍有些不一样。我是这么写的:

@Transacion(value="transManager")
public void s1(){
	update(1);
 	update(2);
}

貌似我从来没有在函数上加过throw MyException,难道是这段有问题?
翻看MyException代码,发现它仅仅继承了Exception。

class MyEception extends Exception {
}

好像就这点不一样。而笔者自定义的Exception基本继承了RuntimeException的。

翻下Spring源码

按照这个思路,笔者去翻了下Spring的源码,看下它在声明式事务中的处理逻辑到底是什么,于是翻到了这一段处理事务异常的代码:

TransactionAspectSupport.java
protected void completeTransactionAfterThrowing(TransactionInfo txInfo, Throwable ex) {
	if (txInfo.transactionAttribute.rollbackOn(ex)) {
	}else{
		// We don't roll back on this exception.
		// Will still roll back if TransactionStatus.isRollbackOnly() is true.
		// 在checkedException的时候,不会被rollBack,会commit!!!
	}
}
@Override
public boolean rollbackOn(Throwable ex) {
	return (ex instanceof RuntimeException || ex instanceof Error);
}

看代码逻辑就明白了,只有异常继承RuntimeException或者Error的时候才会回滚!
好了,让业务开发改了下代码,问题解决了。

总结

遇到问题时,找到出问题代码段和类似的正确代码段的不同处,以此为切入,往往能抓住线索。
日常Bug排查-抛异常不回滚-LMLPHP

06-07 16:28