我创建了一个包含@Transactional注释的Aspect。我的建议是按预期调用的,但是新的实体AuditRecord永远不会保存到数据库中,看起来我的@Transactional注释不起作用。
@Aspect
@Order(100)
public class ServiceAuditTrail {
private AppService appService;
private FooRecordRepository fooRecordRepository;
@AfterReturning("execution(* *.app.services.*.*(..))")
public void logAuditTrail(JoinPoint jp){
Object[] signatureArgs = jp.getArgs();
String methodName = jp.getSignature().getName();
List<String> args = new ArrayList<String>();
for(Object arg : signatureArgs){
args.add(arg.toString());
}
createRecord(methodName, args);
}
@Transactional
private void createRecord(String methodName, List<String> args){
AuditRecord auditRecord = new AuditRecord();
auditRecord.setDate(new Date());
auditRecord.setAction(methodName);
auditRecord.setDetails(StringUtils.join(args, ";"));
auditRecord.setUser(appService.getUser());
fooRecordRepository.addAuditRecord(auditRecord);
}
public void setAppService(AppService appService) {
this.appService = appService;
}
public void setFooRecordRepository(FooRecordRepository fooRecordRepository) {
this.fooRecordRepository= fooRecordRepository;
}
}
bean上下文如下:
<tx:annotation-driven transaction-manager="txManager.main" order="200"/>
<aop:aspectj-autoproxy />
<bean id="app.aspect.auditTrail" class="kernel.audit.ServiceAuditTrail">
<property name="appService" ref="app.service.generic" />
<property name="fooRecordRepository" ref="domain.repository.auditRecord" />
</bean>
我的切入点只拦截接口(服务接口)。服务方法可能是也可能不是交易方式。如果服务方法是事务性的,我希望如果Advice由于某种原因失败,那么该事务将被回滚。
我的问题:为什么忽略事务注释?这是我第一次使用Spring构建AOP服务,我也欢迎任何架构或实现改进。
谢谢!
答案 0 :(得分:15)
在Spring中,@Transactional
通过创建类的代理(Java或cglib代理)并拦截带注释的方法来工作。这意味着如果从同一个类的另一个方法调用带注释的方法,@Transactional
不起作用。
只需将createRecord
方法移动到一个新类(不要忘记将它变成一个Spring bean),它就会起作用。
答案 1 :(得分:3)
一个非常好的问题。如果您必须回滚/提交事务,则可以直接配置Spring事务,如上所述here。
执行此方法不需要手动在每个类/方法上添加@Transactional
。
下面是Spring配置(从参考链接复制)。您只需使用顾问程序/切入点在spring-application context.xml文件中添加配置,而不是编写自定义顾问程序/切入点。
<bean id="fooService" class="x.y.service.DefaultFooService"/>
<!-- the transactional advice (what 'happens'; see the <aop:advisor/> bean below) -->
<tx:advice id="txAdvice" transaction-manager="txManager">
<!-- the transactional semantics... -->
<tx:attributes>
<!-- all methods starting with 'get' are read-only -->
<tx:method name="get*" read-only="true"/>
<!-- other methods use the default transaction settings (see below) -->
<tx:method name="*"/>
</tx:attributes>
</tx:advice>
<!-- ensure that the above transactional advice runs for any execution
of an operation defined by the FooService interface -->
<aop:config>
<aop:pointcut id="fooServiceOperation" expression="execution(* x.y.service.FooService.*(..))"/>
<aop:advisor advice-ref="txAdvice" pointcut-ref="fooServiceOperation"/>
</aop:config>
<!-- don't forget the DataSource -->
<bean id="dataSource" class="org.apache.commons.dbcp.BasicDataSource" destroy-method="close">
<property name="driverClassName" value="oracle.jdbc.driver.OracleDriver"/>
<property name="url" value="jdbc:oracle:thin:@rj-t42:1521:elvis"/>
<property name="username" value="scott"/>
<property name="password" value="tiger"/>
</bean>
<!-- similarly, don't forget the PlatformTransactionManager -->
<bean id="txManager" class="org.springframework.jdbc.datasource.DataSourceTransactionManager">
<property name="dataSource" ref="dataSource"/>
</bean>
REF: