Spring中带代理的包私有方法@Transactional注解是否生效及方案咨询

Spring数据访问文档指出：

• 注册一个将publicMethodsOnly标志设为false的自定义AnnotationTransactionAttributeSource，以在基于类的代理中支持受保护和包私有@Transactional方法。

现有如下代码，想确认其中的@Transactional注解是否生效。若不生效，应注册该自定义AnnotationTransactionAttributeSource，还是将标注注解的方法修改为public？

import org.springframework.transaction.annotation.Transactional;

class ServiceA {
    private final ServiceA self;

    ServiceA(ServiceA self) {
        this.self = self;
    }

    void foo() {
        self.bar();
    }

    @Transactional
    void bar() {
        // ...
    }
}

分析与解答 #

当前代码里的@Transactional注解不生效。Spring默认的AnnotationTransactionAttributeSource只会识别public方法上的事务注解，你的bar()方法是包私有访问权限，即便通过self代理对象调用，默认规则也不会触发事务。

下面是两种可行的解决方式：

方案1：把bar()改为public方法 #

这是Spring官方推荐的标准做法，完全契合事务机制的默认设计。

• 优势：无需额外配置，代码直观易懂，后续维护成本低，不会引入自定义配置带来的潜在冲突。
• 修改后的代码：

@Transactional
public void bar() {
    // ...
}

方案2：自定义AnnotationTransactionAttributeSource #

通过设置publicMethodsOnly=false来支持非public方法的事务，但这种方式属于偏离默认行为的自定义配置。

• 配置示例（Spring Boot环境）：

@Configuration
public class TransactionConfig {
    @Bean
    public TransactionAttributeSource transactionAttributeSource() {
        AnnotationTransactionAttributeSource source = new AnnotationTransactionAttributeSource();
        source.setPublicMethodsOnly(false);
        return source;
    }

    @Bean
    public TransactionInterceptor transactionInterceptor(TransactionAttributeSource transactionAttributeSource, PlatformTransactionManager transactionManager) {
        TransactionInterceptor interceptor = new TransactionInterceptor();
        interceptor.setTransactionAttributeSource(transactionAttributeSource);
        interceptor.setTransactionManager(transactionManager);
        return interceptor;
    }

    @Bean
    public Advisor transactionAdvisor(TransactionInterceptor transactionInterceptor) {
        DefaultPointcutAdvisor advisor = new DefaultPointcutAdvisor();
        advisor.setAdvice(transactionInterceptor);
        advisor.setPointcut(new TransactionAttributeSourcePointcut() {
            @Override
            protected TransactionAttributeSource getTransactionAttributeSource() {
                return transactionAttributeSource();
            }
        });
        return advisor;
    }
}

• 劣势：增加了配置复杂度，团队成员需要额外了解这个自定义规则，还可能和Spring的自动配置产生冲突。

最终建议 #

优先选方案1，把bar()改成public方法。这是最简洁、最符合Spring生态规范的做法，能减少不必要的配置维护工作。

内容的提问来源于stack exchange，提问作者Reddi