我的服务类。
@Service
@Transactional(value = "transactionManager", readOnly = true, propagation = Propagation.REQUIRED)
public class DeviceServiceImpl{
@Transactional(readOnly = false)
public void blockAllDevices(){
createSmsDeviceBlock();
}
public void createSmsDeviceBlock(){
addToLogTables();
smsService.sendSms();
}
@Transactional(readOnly = false,propagation = Propagation.REQUIRES_NEW)
public void addToLogTables(){
try {
//save object with DAO methods...
} catch (Exception e) {
throw new ServiceException(ServiceException.PROCESSING_FAILED, e.getMessage(), e);
}
}
}
从我的控制器,调用服务方法blockAllDevices()。 addToLogTables()方法被标记为Propergation.REQUIRED_NEW,但问题在于addToLogTables()方法没有创建新事务并且现有事务正在使用。
我想要做的是,addToLogTables()方法上的事务应该在执行smsService.sendSms()方法之前提交。
我的问题是,如果事务无法提交,则在方法addToLogTables()方法中,它不应该执行smsService.sendSms()方法。
答案 0 :(得分:9)
这不是Propagation.REQUIRES_NEW
问题。这是@Transactional
代理如何工作的问题。
当Spring代理使用@Transactional
注释的bean时,它基本上将其包装在代理对象中,并在打开事务后委托给它。当委托调用返回时,代理提交或回滚事务。
例如,您的bean是
@Autowired
private DeviceServiceImpl deviceService;
Spring实际上将注入一个包装器代理。
所以当你这样做时
deviceService.blockAllDevices();
您正在调用具有事务行为的代理上的方法。但是,在blockAllDevices()
中,您正在执行
createSmsDeviceBlock();
实际上是
this.createSmsDeviceBlock();
其中this
指的是实际对象,而不是代理,因此没有交易行为。
documentation进一步解释了这一点。
你必须重新设计你的设计。