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

alchemystar 2021-06-07 原文

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

前言

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

Bug现场

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

代码片段

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

乍看上去没啥问题。

思路

笔者用@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的时候才会回滚！
好了，让业务开发改了下代码，问题解决了。