主要原因:事务范围大于锁
@Override
@Transactional
public synchronized CouponsH5Respons getACoupons() {
.....
}
用jmeter进行压力测试的时候,发现优惠券超发
当时很疑惑,synchronized已经放在了方法级别,不应该出现并发的问题
正常情况
- 每个线程的事物都是不同的,T1修改后commit T2读取到新数据
超发场景
- 出现超发的情况,T2 读取到了旧数据
- 可以分析出T2在T1没commit之前读取的数据,因为两者在不同事务,T2读取的一定是旧数据
- 从而分析出synchronized没生效
:
问题原因(事务范围大于锁)
- 因为spring的AOP的特性,会在进入方法之前开启事务,之后再加锁,当锁住的代码执行完成后,再提交事务,如果在T1执行commit之前 有其他线程进来,读取的一定是旧数据
如下图:
同步代码块是在事务的内部
解决方式:
将synchronized关键字加入到Controller层,使synchronized锁的范围大于事务控制的范围。
目的是让同步代码在事务的外面
Object mLock = new Object();
@RequestMapping(value = "test")
@ResponseBody
public void test() throws Exception {
synchronized (mLock) {
service.test();
}
responseMessage(ModelResult.CODE_200,ModelResult.SUCCESS);
}