March 2014 - Topia-devel - mailman.cloud.codelutin.com

Pourquoi les getters ne renverraient pas une collection vide ?
by Brendan Le Ny 24 Nov '14

24 Nov '14

Actuellement, une propriété avec une multiplicité renvoie une collection null si on appelle le getter, pourquoi ne pas initialiser cette collection en lazy (on le fait bien dans le add) ? Voire même, pourquoi pas interdire d'appeler le setter avec null (sauf si Hibernate l'impose) ? Il me semblait y avoir une raison mais elle ne me revient plus. -- Brendan Le Ny, Code Lutin bleny(a)codelutin.com (+33) 02 40 50 29 28

4 4

Pool C3p0 non fermé
by Eric Chatellier 25 Aug '14

25 Aug '14

Bonjoir, Actuellement, lors d'un hot redeploy dans tomcat, le pool c3p0 semble garder les connexions ouvertes. Voici les traces au redémarrage: rave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|6c187ab5]-AdminTaskTimer] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|6c187ab5]-HelperThread-#0] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|6c187ab5]-HelperThread-#1] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|6c187ab5]-HelperThread-#2] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [H2 File Lock Watchdog /tmp/xxx/h2data-web.lock.db] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [H2 Log Writer H2DATA-WEB] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|7dfbfca7]-AdminTaskTimer] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|7dfbfca7]-HelperThread-#0] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|7dfbfca7]-HelperThread-#1] but has failed to stop it. This is very likely to create a memory leak. févr. 19, 2014 10:28:33 PM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads Grave: The web application [/xxx] appears to have started a thread named [C3P0PooledConnectionPoolManager[identityToken->z8kflt90l3muy4yyzl36|7dfbfca7]-HelperThread-#2] but has failed to stop it. This is very likely to create a memory leak. 22:28:36 [ContainerBackgroundProcessor[StandardEngine[Catalina]]] WARN ActiveManagementCoordinator:68 - A C3P0Registry mbean is already registered. This probably means that an application using c3p0 was undeployed, but not all PooledDataSources were closed prior to undeployment. This may lead to resource leaks over time. Please take care to close all PooledDataSources. J'ai trouvé un gros gros hack qui fonctionne: http://stackoverflow.com/a/7472386 ce qui semble bien démontrer qu'il y a un problème quelque part avec Topia/Hibernate (c3p0?). To be continued... -- Éric Chatellier

3 3

Services de migration (NG2 :D)
by Arnaud Thimel 07 Apr '14

07 Apr '14

Salut, On a relancé cet aprem un vieux débat sur le service de migration. Comme on était en nombre réduit, je tente de synthétiser les différentes idées qui en sont ressorties, dîtes-nous ce que vous en pensez. J'ajoute également que j'ai pris certaines libertés dans la rédaction de ce mail sans consulter Brendan et Éric avec qui j'en ai discuté cet aprem. Proposition 1 : distinguer la version de l'application de la version de la base de données ========================================================================================== Exemple : * Version de l'application = 0.10.3 * Versions de la base = 4, 5, 6 La version de la base serait un entier tout con qu'on incrémente à chaque modification de base. Pour les versions ultérieures, on aurait : * Version de l'application = 0.10.4 * Versions de la base = 7 Puis : * Version de l'application = 0.11 * Versions de la base = 8, 9 etc... Proposition 2 : une classe de migration par version applicative =============================================================== La classe Java de migration (une seule par version applicative) serait donc un regroupement logique des migrations de version de base liées à cette version (applicative). Si je reprends l'exemple, la classe MachinCallbackV0_10_3contiendrait 3 méthodes, une pour chaque version de base (migrateToVersion4, migrateToVersion5 et migrateToVersion6). Cette classe serait responsable du choix de commit, au choix parmi : - après chaque version de base ; - après l'ensemble des versions de base qui constituent la version applicative. Au passage il serait cool de pouvoir aller lire et exécuter un fichier "migrateToVersion5.sql" en lieu et place d'un appel à la méthode Java. Proposition 3 : conserver un historique dans la table TMSVersion ================================================================ Aujourd'hui, la table TMSVersion ne contient qu'une seule et unique ligne avec une seule colonne qui contient "0.10.2": topiaId | version ----------------- xxxxx | 0.10.2 La nouvelle table contiendrait plus d'informations (exemple) : topiaId | dbVersion | appVersion | date | success | comment ---------------------------------------------------------------------------------- xxxxx | 5 | 0.10.3 | 21/11/2013 | true | suppr de la table machin xxxxx | 6 | 0.10.3 | 21/11/2013 | true | renommage colonne truc xxxxx | 7 | 0.10.4 | 25/12/2013 | false | ajout table pouette xxxxx | 7 | 0.10.4 | 26/12/2013 | true |ajout table pouette (retry) xxxxx | 8 | 0.11 | 01/01/2014 | true | - xxxxx | 9 | 0.11 | 01/01/2014 | true | - Pour connaître la version actuelle de la base, cette simple requête suffit : select max(dbVersion) from tmsversion where success is true; Ce ne sont que des propositions que j'ai présenté ici, il y aura certainement des points bloquants (migration pour les applis existantes, ...) auxquels nous nous attaquerons si ces propositions conviennent au plus grand nombre. À vous. Cordialement, Arnaud

3 2

Modification de la session factory
by Eric Chatellier 07 Mar '14

07 Mar '14

Salut, Pour info, je suis en train de revoir le fonctionnement des sessionFactory/metadatasources/serviceregistry dans Topia. J'ai commencer par supprimer la Configuration (petit détail :) ) car elle n'est pas encore déprécié, mais c'est tout comme : https://docs.jboss.org/hibernate/core/4.3/javadocs/org/hibernate/cfg/Config… Je pense qu'on instancie déjà plusieurs ServiceRegistry qui peuvent poser problème. Je vais essayer dans laisser un seul. J'ai réussi à chargé les mappings avec la metadatasources, mais j'ai identifié un bug bloquant que je dois remonter à hibernate. To be continued... -- Éric Chatellier - www.codelutin.com - 02.40.50.29.28

2 5