r2821 - trunk/topia-persistence/src/main/java/org/nuiton/topia/framework
Author: tchemit Date: 2013-10-10 15:09:17 +0200 (Thu, 10 Oct 2013) New Revision: 2821 Url: http://nuiton.org/projects/topia/repository/revisions/2821 Log: do not close topia context when GC Modified: trunk/topia-persistence/src/main/java/org/nuiton/topia/framework/AbstractTopiaContext.java Modified: trunk/topia-persistence/src/main/java/org/nuiton/topia/framework/AbstractTopiaContext.java =================================================================== --- trunk/topia-persistence/src/main/java/org/nuiton/topia/framework/AbstractTopiaContext.java 2013-10-10 13:04:42 UTC (rev 2820) +++ trunk/topia-persistence/src/main/java/org/nuiton/topia/framework/AbstractTopiaContext.java 2013-10-10 13:09:17 UTC (rev 2821) @@ -933,15 +933,24 @@ /** * Pour le context root on ferme tous les fils, et la factory hibernate. + * + * <strong>Note:</strong> Non, il n'est pas de la responsabilité du GC de + * fermer de telles ressources, les devleoppeurs doivent gérer ça à la main... */ @Override protected void finalize() throws Throwable { - if (hibernateFactory != null) { - closeContext(); - hibernateFactory.close(); - closed = true; - log.debug("TopiaContext finalized"); +// if (hibernateFactory != null) { +// closeContext(); +// hibernateFactory.close(); +// closed = true; +// log.debug("TopiaContext finalized"); +// } + if (!closed) { + + // means resource was not well closed, can't accept this! + throw new TopiaException("TopiaContext " + this + " was not closed!"); } + super.finalize(); } @Override
participants (1)
-
tchemit@users.nuiton.org