Re: [Topia-devel] [Topia-commits] r2883 - trunk/topia-persistence/src/main/java/org/nuiton/topia
On Fri, 22 Nov 2013 18:48:56 +0100 (CET) athimel@users.nuiton.org wrote:
Author: athimel Date: 2013-11-22 18:48:55 +0100 (Fri, 22 Nov 2013) New Revision: 2883
Url: http://nuiton.org/projects/topia/repository/revisions/2883
Log: fixes #2922 ApplicationContext now keeps a weak reference on child PersistenceContexts
Modified: trunk/topia-persistence/src/main/java/org/nuiton/topia/AbstractTopiaApplicationContext.java
Modified: trunk/topia-persistence/src/main/java/org/nuiton/topia/AbstractTopiaApplicationContext.java =================================================================== --- trunk/topia-persistence/src/main/java/org/nuiton/topia/AbstractTopiaApplicationContext.java 2013-11-22 16:38:04 UTC (rev 2882) +++ trunk/topia-persistence/src/main/java/org/nuiton/topia/AbstractTopiaApplicationContext.java 2013-11-22 17:48:55 UTC (rev 2883) @@ -26,16 +26,12 @@
AMHA quelque chose ne va pas dans ce code, il faut qu'on essaye de tout fermer, donc si y'a un pb sur la fermeture d'un context fils, on trace l'erreur et on doit continuer à fermer les autres fils.
+ // Iterate over the children PersistenceContexts and close them + for (TopiaPersistenceContext persistenceContext : persistenceContexts) { + if (persistenceContext == null) { + if (log.isWarnEnabled()) { + log.warn("null TopiaPersistenceContext found in #persistenceContexts"); + } + } else { + // Avoid to have exception from checkClosed method on child + if (!persistenceContext.isClosed()) { + persistenceContext.closeContext(); + } + } + }
hibernateProvider.close(); closed = true;
_______________________________________________ Topia-commits mailing list Topia-commits@list.nuiton.org http://list.nuiton.org/cgi-bin/mailman/listinfo/topia-commits
-- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 http://www.codelutin.com email: chemit@codelutin.com twitter: https://twitter.com/tchemit
Le 22/11/2013 21:21, Tony Chemit a écrit :
AMHA quelque chose ne va pas dans ce code, il faut qu'on essaye de tout fermer, donc si y'a un pb sur la fermeture d'un context fils, on trace l'erreur et on doit continuer à fermer les autres fils. Oui, j'avoue avoir hésité là dessus, je complète.
Arnaud
Le 22/11/2013 22:15, Arnaud Thimel a écrit :
Le 22/11/2013 21:21, Tony Chemit a écrit :
AMHA quelque chose ne va pas dans ce code, il faut qu'on essaye de tout fermer, donc si y'a un pb sur la fermeture d'un context fils, on trace l'erreur et on doit continuer à fermer les autres fils. Oui, j'avoue avoir hésité là dessus, je complète.
J'me suis autorisé un "catch (Exception)", histoire d'être sûr que rien ne vienne empêcher la fermeture des contextes... Arnaud
participants (2)
-
Arnaud Thimel -
Tony Chemit