ChatGPT解决这个技术问题 Extra ChatGPT

@Transactional method calling another method without @Transactional anotation?

I've seen a method in a Service class that was marked as @Transactional, but it was also calling some other methods in that same class which were not marked as @Transactional.

Does it mean that the call to separate methods are causing the application to open separate connections to DB or suspend the parent transaction, etc?

What's the default behavior for a method without any annotations which is called by another method with @Transactional annotation?


4
4javier

When you call a method without @Transactional within a transaction block, the parent transaction will continue to the new method. It will use the same connection from the parent method (with @Transactional) and any exception caused in the called method (without @Transactional) will cause the transaction to rollback as configured in the transaction definition.

If you call a method with a @Transactional annotation from a method with @Transactional belonging to the same Spring Bean, then the called methods transactional behavior will not have any impact on the transaction. But if you call a method with a transaction definition from another method with a transaction definition, and they belong to different Spring Beans, then the code in the called method will follow its own transaction definitions.

You can find more details in the section Declarative transaction management of spring transaction documentation.

Spring declarative transaction model uses AOP proxy, so the AOP proxy is responsible for transactions creation. The AOP proxy will be active only if the called method belong to a different Spring Bean than the caller one.


is that the spring default behavior ?
@Tomasz Yes. But it should also be mentioned that changing transaction propagation on a method that is called from another @Transactional method will have no effect.
@Tomasz, That was what I meant by saying will follow the transaction definitions given in the called method. But if the call is comes from the same object instance it will not have any effect since the call will not propagate through the aop proxies which are responsible for the transaction maintenance.
@Filip, That is not completely correct, If you call a method with a @Transactional definition from a different object/instance then even though the calling method have a different @Transactional attributes, the called method will follow the it own transaction definition.
@ArunPJohny what does it mean: "within the same instance"? Eg. I have parent method with Transactional annotation and within this method I call another method with Transactional from another service... When in child method something woll go wrong with exception, then I want to rollback everything...
t
tolitius

Does that mean the call to separate methods are causing the application to open separate connections to DB or suspend the parent transaction, etc?

That depends on a propagation level. Here are all the possible level values.

For example in case a propagation level is NESTED a current transaction will "suspend" and a new transaction will be created ( note: actual creation of a nested transaction will only work on specific transaction managers )

What's the default behavior for a method without any annotations that is called by another method with @Transactional annotation?

The default propagation level ( what you call "behavior" ) is REQUIRED. In case an "inner" method is called that has a @Transactional annotation on it ( or transacted declaratively via XML ), it will execute within the same transaction, e.g. "nothing new" is created.


What about subcalls of NOT_SUPPORTED which dont have any annotation? Does it inherit NOT_Supported or did they open a new transaction as REQURED is the default? For example: f1.call(){ f2() } with the annotation NOT_SUPPORTED for f1 and non for f2.
s
sourcedelica

@Transactional marks the transaction boundary (begin/end) but the transaction itself is bound to the thread. Once a transaction starts it propagates across method calls until the original method returns and the transaction commits/rolls back.

If another method is called that has a @Transactional annotation then the propagation depends on the propagation attribute of that annotation.


The 3 answers conflict with each other in some degree, not sure which is more accurate.
@EricWang Just wanted to share that I tested out this scenario today and the answer by Arun P Johny (with comments) is the most accurate for this scenario of internal invocations.
C
C-Otto

The inner method will affect the outer method if the inner method is not annotated with @Transactional.

In case inner method is also annotated with @Transactional with REQUIRES_NEW, following will happen.

...
@Autowired
private TestDAO testDAO;

@Autowired
private SomeBean someBean;

@Override
@Transactional(propagation=Propagation.REQUIRED)
public void outerMethod(User user) {
  testDAO.insertUser(user);
  try{
    someBean.innerMethod();
  } catch(RuntimeException e){
    // handle exception
  }
}


@Override
@Transactional(propagation=Propagation.REQUIRES_NEW)
public void innerMethod() {
  throw new RuntimeException("Rollback this transaction!");
}

The inner method is annotated with REQUIRES_NEW and throws a RuntimeException so it will set its transaction to rollback but WILL NOT EFFECT the outer transaction. The outer transaction is PAUSED when the inner transaction starts and then RESUMES AFTER the inner transaction is concluded. They run independently of each other so the outer transaction MAY commit successfully.


To clarify for beginners, I'm pretty sure innerMethod() needs to be on a different bean (aka Spring-managed java Object) than outerMethod(). If they are both on the same bean, I don't think the innerMethod will actually use the Transactional behavior declared in its annotation. Rather it will use what is declared in the outerMethod() declaration. This is because of how Spring handles AOP, which is used for it's @Transactional annotations (docs.spring.io/spring/docs/3.0.x/spring-framework-reference/…)