views:

147

answers:

2

We have:

@Transactional(propagation = Propagation.REQUIRED)
public class MyClass implementes MyInterface { ...

MyInterface has a single method: go().

When go() executes we start a new transaction which commits/rollbacks when the method is complete - this is fine.

Now let's say in go() we call a private method in MyClass that has @Transactional(propagation = Propagation.REQUIRES_NEW. It seems that Spring "ignores" the REQUIRES_NEW annotation and does not start a new transaction. I believe this is because Spring AOP operates on the interface level (MyInterface) and does not intercept any calls to MyClass methods. Is this correct?

Is there any way to start a new transaction within the go() transaction? Is the only way to call another Spring managed bean that has transactions configured as REQUIRES_NEW?


Update: Adding that when clients execute go() they do so via a reference to the interface, not the class:

@Autowired
MyInterface impl;

impl.go();
+4  A: 

@Transactional will only be noticed if it's on a public method, due to the way Spring AOP works.

However, you can programmatically start a new transaction if you desire, using TransactionTemplate, e.g.

TransactionTemplate txTemplate = new TransactionTemplate(txManager);                
txTemplate.setPropagationBehavior(TransactionDefinition.PROPAGATION_REQUIRES_NEW);
txTemplate.execute(new TransactionCallback<Object>() {
    public Object doInTransaction(TransactionStatus status) {
        // do stuff
    }
});
skaffman
But even if `@Transactional` is on a public method of MyClass, it still appears that Spring won't pick it up **unless** the method is defined in the interface - correct?
Marcus
To add, this is because the client that is executing the go() method in question has a reference to the interface, not the class.
Marcus
@Marcus: Sort of. If `MyClass` implements an interface, then Spring will use only that interface for generating the transactional proxy, and ignore even public, non-interface methods. However, if `MyClass` implements no interface, then all public methods will be used.
skaffman
+2  A: 

From the Spring reference 2.5:

When using proxies, the @Transactional annotation should only be applied to methods with public visibility. If you do annotate protected, private or package-visible methods with the @Transactional annotation, no error will be raised, but the annotated method will not exhibit the configured transactional settings.

So Spring ignores @Transactional annotation on non-public methods.

Also,

In proxy mode (which is the default), only 'external' method calls coming in through the proxy will be intercepted. This means that 'self-invocation', i.e. a method within the target object calling some other method of the target object, won't lead to an actual transaction at runtime even if the invoked method is marked with @Transactional!

So even if you make your method public, calling it from within a method of same class will not start a new transaction.

You can use aspectj mode in transaction settings so that the transaction related code is weaved in the class and no proxy is created at runtime.

See the reference document for more details.

abhin4v