4

Does CDI @Transactional(Transactional.TxType.REQUIRES_NEW) works when is called inside the same bean:

@Transactional
public void method1() {
    for(...) {
        method2();
    }
}

@Transactional(Transactional.TxType.REQUIRES_NEW)
public void method2() {
   ...
}

I suppose it doesn't works because local calls can't be intercepted by the proxy. In that case what is the recommeded usage pattern if I want to manage each call of method2 in a new transaction. The obvious one is to create another bean, but it's something I don't like.

lujop
  • 13,504
  • 9
  • 62
  • 95
  • 2
    If you can use EJB for your controllers instead of CDI you will be better off at this point. EJBs are just as simple, more mature and don't have issues like not being able to self reference or call local methods. With EJB you can either inject a ref of the same bean or use the EJB session context to call the local method. – NBW Mar 21 '15 at 19:54
  • 1
    @NBW But the issue with local methods it's the same, isn't it? The solution will be to obtain the Bussiness interface and call the local method? – lujop Mar 22 '15 at 08:17
  • 1
    I wonder why you actually want to open another transaction? I don't see a `try`/`catch` in your `method1` What happens if one of the `method2` transactions fails? Is the transaction surrounding `method1` really still valid? Usually transactions give you an all-or-nothing semantics, and with your above code snippet I struggle to see the benefit of a nested transaction. FWIW, by the SRP you most probably still want to move `method2` to another bean, *in particular* if its value/use-case even justifies a new transaction. – Alexander Langer Mar 22 '15 at 10:06

1 Answers1

5

Interception of local method calls does not work for CDI and you cannot inject your class directly into itself (cyclic dependency). See also CDI call interceptor annotated method within same instance

The self injection issue is still open, see https://issues.jboss.org/browse/CDI-414

Community
  • 1
  • 1
mp911de
  • 17,546
  • 2
  • 55
  • 95