Discussion :

[Gregouze]

J'ai un EJB Session Stateful avec transaction de type Container et je voulais savoir si ceci est réalisable:

Voici une méthode de cet EJB:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
try {
     blablabla
}
catch (Exception e) {
     doSomething()
}
     doSomethingElse()

Lorsque aucune exception est levée, tout ce qui est fait dans doSomethingElse est commité c'est le fonctionnement normal du Container.

Lorsque une exception est levée je voudrais que tout ce qui est fait dans le catch (le doSomething) soit lui commité.

Le probleme c'est que aujourd'hui toute ma transaction est rollbackée.

[xv-mnt]

A mon avis, tu ne catche pas l'exception. Dans le code présenté tu catche tout ce qui est de type Exception. Donc toutes les RuntimeException (NullPointer, etc...) ne sont pas catchées par ton code mais par le container qui réalise alors un rollback.
Change Exception par Throwable, et regarde si çà marche.

[Gregouze]

Alors en fait je n'ai pas été complet:

voici plutot la portion de code que j'execute:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
try {  
        doSomethingElse()
 } catch (IllegalStateException e) {  
        doSomething()
        throw e;
}

Dans le catch je renvoie l'exception dans l'idée justement de rollbacker tout ce qui a pu etre fait en amont ... le fond de la question est donc de savoir si il existe un moyen pour que tout ce qui est fait dans doSomething soit commité tout en rollbackant ce qui a pu etre fait dans le doSomethingElse ... hmm hmm je sais pas si c'est tres clair ...

[lanfeust23]

En mettant ton doSomething() dans un finally par exemple ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
boolean flag=false
try { 
doSomethingElse()
} catch (IllegalStateException e) {
flag=true;
throw e; 
}
finally{if(flag)  doSomething() ;}

[Gregouze]

Non ca ne marche pas mieux ...

Le doSomething() contient des ordres d'update dans une base de donnée mais je pense que la transaction est marquée rollbackée à partir du moment ou l'exception est levée et donc tout ce qui est fait dans le doSomething est lui aussi rollbacké ...

L'orde d'update dans le doSomething leve cette trace:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
[TxInterceptorCMT] IllegalStateException while setting transaction for rollback only
java.lang.IllegalStateException: Already rolled back. TransactionImpl:XidImpl[FormatId=257, GlobalId=fr-31-01-05-072/17, BranchQual=, localId=17]
	at org.jboss.tm.TransactionImpl.setRollbackOnly(TransactionImpl.java:691)

[manblaizo]

Et si tu indiquais un "RequiresNew" comme attribut de transaction pour la méthode doSomething(), qu'est-ce que ça donne ? Parce que dans ce cas, la méthode doSomething() s'exécuterait dans une autre transaction, commiterait, puis rendrait la main pour relancer l'autre transaction rollbackée. Bon, j'ai pas encore vérifié ça, si tu peux nous dire ce qu'il en retourne.

[Gregouze]

Effectivement c'est une piste que j'ai essayé d'explorer ...
Voici ce que j'ai (je travaille avec XDoclet):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
/**
 * 
 * <!-- begin-xdoclet-definition -->
 * 
 * @ejb.interface-method view-type="remote" <!-- end-xdoclet-definition -->
 * @generated
 * 
 */
 
public void method1 {
  try { 
     doSomethingElse()  
  } 
  catch (IllegalStateException e) { 
     doSomething()  
     throw e; 
  }
}

avec doSomething qui est une methode de ce meme EJB:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
/**
 * @ejb.interface-method view-type="local" <!-- end-xdoclet-definition -->
 * @ejb.transaction type="RequiresNew"
*/
public void doSomething(){
     //appel d'un EJB Session facade qui lui meme fait un save en base via Hibernate
}

Et la transaction est qd meme rolbackée!!!
Ceci dit ca ne m'étonne qu'a moitié puisque la transaction de haut niveau c'est à dire celle de la méthode "method1" est rollbackée ...
J'ai l'impression que je me marche sur les pieds ...

[OButterlin]

Et pourquoi ne pas faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
try
{
   doSomethingElse();
   "commit()";  
}
catch (Throwable t)
{
   "rollback()";
   doSomething();
   "commit()";
   throw t;
}

A+

[Gregouze]

En fait je ne peux pas faire ce que tu dis puisque je suis dans un Session bean en mode Container Managed Transaction.

J'ai trouvé visiblement la solution.

En fait le RequiresNew était une bonne solution mais il me restait un problème à savoir que la méthode que j'appelais faisait partie du même session bean ...

En créant un nveau session bean en lui ajoutant la fameuse doSomething en RequiresNew, cela marche ...

Le lien suivant fournissait en quelques sorte la réponse:
http://www.jboss.com/index.html?modu...opic&p=3807209

Donc au final la solution suivante marche:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
/**
 * 
 * <!-- begin-xdoclet-definition -->
 * 
 * @ejb.interface-method view-type="remote" <!-- end-xdoclet-definition -->
 * @generated
 * 
 */
 
public void method1 {
  try { 
     ejb1.doSomethingElse()  
  } 
  catch (IllegalStateException e) { 
     ejb2.doSomething()  
     throw e; 
  }
}

avec doSomething en transition type = RequiresNew.