EntityManager Annotation performance

Et oui, c'est encore moi, un problème résolu et en voici déjà un autre qui pointe le bout de son nez :aie:

Voilà, j'utilise Hibernate dans une application web avec Servlet et jsp . Pour Hibernate j'utilise l'EntityManager, les Annotations et le fichier persistence.xml.

J'ai l'impression d'avoir des problèmes de performance.
J'ai lu sur pas mal de site qu'il fallait gérer les problèmes de Thread, mais sans trouver quelquechose de précis sur la manière de faire.

Pour le moment j'ai une classe EntityManagerUtil qui gère la création et la fermeture de l'entityManager. Mais j'ai pas bien compris comment l'utiliser au niveau de ma servlet (j'en ai meme plusieurs).

Si vous avez une idée de comment on fait, ou bien des sites où c'est expliqué, je suis preneuse.

Merci beaucoup.

Pour améliorer les performances de l'utilisation d'Hibernate, il faut surtout maîtriser la manière dont tu récupères tes objets en fonction de ce que tu vas en faire : est-ce qu'il vaut mieux utiliser le lazy loading ? faire des sous-requêtes ? au contraire remonter tous les objets ?
L'utilisation du cache peut aussi être utile, pour les données qui changent rarement.
Il y a des explications ici :
http://www.hibernate.org/hib_docs/re...rformance.html

Quand je parlais de performance, je faisais allusion à ce qui est décrit dans ce site
http://jl2tho.blogspot.com/2007/08/t...ityManagerUtil

Il utilise leur propre classe "WebContainerServlet" pour leur Servlet.
Et je ne souhaite pas l'utiliser.

Est ce que ce que décrit ce site est pertinent, ou bien vaut il mieux que je créé pour chaque classe ou je fais mes traitements un EntityManagerFactory, EntityManager et une Transaction.

L'objectif de cette classe c'est que l'objet EntityManager (ou l'objet Session, c'est comme tu veux...) ne soit pas créé à chaque fois qu'on en a besoin. Ce qui nuirait à la performance.

Ici l'objet est créé la première fois qu'on en a besoin. Et il n'est pas détruit.
Moi j'utilise la classe suivante :

HibernateUtil.java

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

import org.hibernate.*; import org.hibernate.cfg.*;   public class HibernateUtil {   private static final SessionFactory sessionFactory;   static { try { sessionFactory = new AnnotationConfiguration().configure().buildSessionFactory(); } catch (Throwable ex) { throw new ExceptionInInitializerError(ex); } }   public static Session getSession() throws HibernateException { return sessionFactory.openSession(); } }

---

Et j'y fais appel de la façon suivante :

Code:

---

1 2	Session session = HibernateUtil.getSession();

---

Est-ce que cela répond à ta question ?

D'accord donc je garde ma classe EntityManagerUtil
qui est comme ça. Est ce que ça te semble bon?

Citation:

---

package abaque.traitement;

import javax.persistence.EntityManager;
import javax.persistence.EntityManagerFactory;
import javax.persistence.Persistence;

import org.apache.log4j.Logger;

public class EntityManagerUtil {

private static final Logger log = Logger
.getLogger(SiteServlet.class);

private static EntityManagerFactory emf;
public static final ThreadLocal<EntityManager> entitymanager = new ThreadLocal<EntityManager>();

public static EntityManagerFactory getEntityManagerFactory(){
if (emf == null){
log.info("EntityManagerUtil.getEntityManagerFactory : nouveau EntityManagerFactory");

// Create the EntityManagerFactory
// Attention : il s'agit du nom de la PersistenceUnit du fichier persistence.xml qui doit etre utilise
emf = Persistence.createEntityManagerFactory(Constantes.PERSISTENCE_NAME);
}
return emf;
}

public static EntityManager getEntityManager(){
EntityManager em = entitymanager.get();

//Create a new EntityManager
if (em == null){
em = emf.createEntityManager();
entitymanager.set(em);
}
return em;
}

/**

* Ferme l'entityManager afin d'assurer l'isolation.

*/

public static void closeEntityManager() {
EntityManager em = entitymanager.get();
entitymanager.set(null);

if (em != null) em.close();

}
}

---

J'ai une dernière question
As tu rajouté du code au niveau de ta Servlet? (Sachant qu'en plus moi j'utilise plusieurs Servlet)
Moi par exemple j'ai surchargé les classes init() et destroy()

La classe m'a l'air bien.

Sauf que je ne voie bien l'intérêt de la méthode :closeEntityManager()
Puisque l'objectif c'est de garder l'EntityManager ouvert et ne pas avoir à le recréer avant chaque requête...

Quand tu as besoin de l'entityManager :

Code:

---

1 2	EntityManager entityManager = EntityManagerUtil.getEntityManager ();

---

Je ne comprend pas ce que tu veux dire par rapport aux méthodes init et destroy. Peux-tu montrer un exemple ?

En gros dans ma servlet, j'ai ça :

Citation:

---

public void destroy() {
log.info("Site.destroy");
System.out.println("Site.destroy");
EntityManagerUtil.closeEntityManager();
super.destroy();
}

public void init() throws ServletException {
super.init();
log.info("Site.init");
System.out.println("Site.init");
EntityManagerUtil.getEntityManagerFactory();
}

---

Est ce vraiment utile? Je ne sais pas trop en fait.

En fait ce qui m'inquiète c'est que dans les logs, j'ai souvent ce message qui apparait et j'ai pensé que cela avait peut être un rapport (à cause de l'histoire du thread et par rapport au site que j'ai mis plus haut :

Citation:

---

Exception in thread "Timer-6" java.lang.NullPointerException
at com.mchange.v2.log.log4j.Log4jMLog$Log4jMLogger.isLoggable(Log4jMLog.java:252)
at com.mchange.v2.resourcepool.BasicResourcePool$CullTask.run(BasicResourcePool.java:1285)
at java.util.TimerThread.mainLoop(Timer.java:512)
at java.util.TimerThread.run(Timer.java:462)

---

Est-ce que tu as la même erreur en supprimant la ligne :

Code:

---

EntityManagerUtil.closeEntityManager();

---

de la méthode destroy() ?

Oui j'ai la même erreur.
En plus je ne sais pas exactement quand elle est lancée (l'erreur)

Dans la méthode init() la ligne suivante ne sert à rien :

Code:

---

EntityManagerUtil.getEntityManagerFactory();

---

Tu récupère l'objet mais tu n'en fais rien...

Il faut que tu l'enregistre (par exemple) comme un objet de ta classe

Code:

---

maFactory = EntityManagerUtil.getEntityManagerFactory();

---

Par contre je n'ai pas d'idée concernant le nullPointerException...

en effet "EntityManagerUtil.getEntityManagerFactory();" dans le init()
ça instancie un EntityManagerFactory au niveau de EntityManagerUtil, si je l'enleve de là, après mon application plante. Il faut dans ce cas que je rajoute cette ligne dans chaque classe qui execute mes traitements hibernate?