spring transaction management Propagation.REQUIRES_NEW无效

时间:2014-01-28 14:13:49

标签: spring transactions required propagation requires

我的服务类。

@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()方法。

1 个答案:

答案 0 :(得分:9)

这不是Propagation.REQUIRES_NEW问题。这是@Transactional代理如何工作的问题。

当Spring代理使用@Transactional注释的bean时,它基本上将其包装在代理对象中,并在打开事务后委托给它。当委托调用返回时,代理提交或回滚事务。

例如,您的bean是

@Autowired
private DeviceServiceImpl deviceService;

Spring实际上将注入一个包装器代理。

所以当你这样做时

deviceService.blockAllDevices();

您正在调用具有事务行为的代理上的方法。但是,在blockAllDevices()中,您正在执行

createSmsDeviceBlock();

实际上是

this.createSmsDeviceBlock();

其中this指的是实际对象,而不是代理,因此没有交易行为。

documentation进一步解释了这一点。

你必须重新设计你的设计。