4

我的服务类。

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

4

1 回答 1

11

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

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

例如，您的 bean 是

@Autowired
private DeviceServiceImpl deviceService;

Spring 实际上是要注入一个包装代理。

所以当你这样做时

deviceService.blockAllDevices();

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

createSmsDeviceBlock();

这实际上是

this.createSmsDeviceBlock();

wherethis指的是实际对象，而不是代理，因此没有事务行为。

这在文档中有进一步的解释。

你将不得不重新设计你的设计。

于 2014-01-28T14:54:08.487 回答