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