[Hibernate] Exception session is closed

Version imprimable

J'ai un problème bizarre avec Hibernate que je n'avais pas auparavant avec les même methodes. Dès que j'utilise une methode du type :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
public User recupererUser(int idUser) {
		Session s = HibernateUtil.currentSession();
		Transaction tx = s.beginTransaction();
 
		User us = (User) s.get(User.class,idUser);
 
		tx.commit();
		HibernateUtil.closeSession();
		return us;		
 
	}

J'obtiens une exception

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
 
INFO: Error performing load command
org.hibernate.SessionException: Session is closed!
	at org.hibernate.impl.AbstractSessionImpl.errorIfClosed(AbstractSessionImpl.java:49)
	at org.hibernate.impl.SessionImpl.getBatcher(SessionImpl.java:255)
	at org.hibernate.loader.Loader.doQuery(Loader.java:714)
	at org.hibernate.loader.Loader.doQueryAndInitializeNonLazyCollections(Loader.java:224)
	at org.hibernate.loader.Loader.loadEntity(Loader.java:1785)
	at org.hibernate.loader.entity.AbstractEntityLoader.load(AbstractEntityLoader.java:47)
	at org.hibernate.loader.entity.AbstractEntityLoader.load(AbstractEntityLoader.java:41)
	at org.hibernate.persister.entity.AbstractEntityPersister.load(AbstractEntityPersister.java:2730)
	at org.hibernate.event.def.DefaultLoadEventListener.loadFromDatasource(DefaultLoadEventListener.java:365)
	at org.hibernate.event.def.DefaultLoadEventListener.doLoad(DefaultLoadEventListener.java:346)
	at org.hibernate.event.def.DefaultLoadEventListener.load(DefaultLoadEventListener.java:123)
	at org.hibernate.event.def.DefaultLoadEventListener.proxyOrLoad(DefaultLoadEventListener.java:177)
	at org.hibernate.event.def.DefaultLoadEventListener.onLoad(DefaultLoadEventListener.java:87)
	at org.hibernate.impl.SessionImpl.fireLoad(SessionImpl.java:862)
	at org.hibernate.impl.SessionImpl.get(SessionImpl.java:799)
	at org.hibernate.impl.SessionImpl.get(SessionImpl.java:792)
	at persistance.HibernateManager.recupererUser(HibernateManager.java:111)
	at metier.GestionUser.recupererUser(GestionUser.java:163)
	at controleur.ServletFichier.doPost(ServletFichier.java:189)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:709)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:252)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:213)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:178)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:126)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:105)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:107)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:148)
	at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:869)
	at org.apache.coyote.http11.Http11BaseProtocol$Http11ConnectionHandler.processConnection(Http11BaseProtocol.java:664)
	at org.apache.tomcat.util.net.PoolTcpEndpoint.processSocket(PoolTcpEndpoint.java:527)
	at org.apache.tomcat.util.net.LeaderFollowerWorkerThread.runIt(LeaderFollowerWorkerThread.java:80)
	at org.apache.tomcat.util.threads.ThreadPool$ControlRunnable.run(ThreadPool.java:684)
	at java.lang.Thread.run(Unknown Source)
16 juin 2006 14:39:32 org.apache.catalina.core.StandardWrapperValve invoke
GRAVE: "Servlet.service()" pour la servlet ServletFichier a généré une exception
org.hibernate.SessionException: Session is closed!
	at org.hibernate.impl.AbstractSessionImpl.errorIfClosed(AbstractSessionImpl.java:49)
	at org.hibernate.impl.SessionImpl.getBatcher(SessionImpl.java:255)
	at org.hibernate.loader.Loader.doQuery(Loader.java:714)
	at org.hibernate.loader.Loader.doQueryAndInitializeNonLazyCollections(Loader.java:224)
	at org.hibernate.loader.Loader.loadEntity(Loader.java:1785)
	at org.hibernate.loader.entity.AbstractEntityLoader.load(AbstractEntityLoader.java:47)
	at org.hibernate.loader.entity.AbstractEntityLoader.load(AbstractEntityLoader.java:41)
	at org.hibernate.persister.entity.AbstractEntityPersister.load(AbstractEntityPersister.java:2730)
	at org.hibernate.event.def.DefaultLoadEventListener.loadFromDatasource(DefaultLoadEventListener.java:365)
	at org.hibernate.event.def.DefaultLoadEventListener.doLoad(DefaultLoadEventListener.java:346)
	at org.hibernate.event.def.DefaultLoadEventListener.load(DefaultLoadEventListener.java:123)
	at org.hibernate.event.def.DefaultLoadEventListener.proxyOrLoad(DefaultLoadEventListener.java:177)
	at org.hibernate.event.def.DefaultLoadEventListener.onLoad(DefaultLoadEventListener.java:87)
	at org.hibernate.impl.SessionImpl.fireLoad(SessionImpl.java:862)
	at org.hibernate.impl.SessionImpl.get(SessionImpl.java:799)
	at org.hibernate.impl.SessionImpl.get(SessionImpl.java:792)
	at persistance.HibernateManager.recupererUser(HibernateManager.java:111)
	at metier.GestionUser.recupererUser(GestionUser.java:163)
	at controleur.ServletFichier.doPost(ServletFichier.java:189)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:709)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:252)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:213)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:178)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:126)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:105)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:107)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:148)
	at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:869)
	at org.apache.coyote.http11.Http11BaseProtocol$Http11ConnectionHandler.processConnection(Http11BaseProtocol.java:664)
	at org.apache.tomcat.util.net.PoolTcpEndpoint.processSocket(PoolTcpEndpoint.java:527)
	at org.apache.tomcat.util.net.LeaderFollowerWorkerThread.runIt(LeaderFollowerWorkerThread.java:80)
	at org.apache.tomcat.util.threads.ThreadPool$ControlRunnable.run(ThreadPool.java:684)
	at java.lang.Thread.run(Unknown Source)

et a chaque fois la ligne incrimminée est la ligne ou je tente d'effectuer un acces à la base avec la session obtenu par HibernateUtil.

La je ne vois pas du tout comment resoudre mon souci. Qu'est qui peut fermer la connexion entre le moment ou je la récupère et celui ou je veux m'en servir.

16/06/2006, 14h47
cicolas

ça peut venir dta factoty créée par HibernateUtil, qui sera pas instanciée, ou fermé ! du coup, il peut pas faire d'action sur la session récupérée avec !
Non? ;)
16/06/2006, 15h02
K-Kaï

Non il me retourne bien une session convenable, a vrai dire même sur l'instruction juste avant le s.get() par exemple, la variable session est bien instancié et son paramètre closed est bien à false (verifié avec le debugger).

Je ne sais même pas quoi chercher en faite c'est ca le problème ...
16/06/2006, 18h50
Nesmontou

Salut, le problème doit sûrement venir du fait que tu utilises HibernateUtil.closeSession(); qui ferme ta session ; le premier appel à une de ces méthodes doit fonctionner mais les suivants vont échouer puisque tu as fermé la session. Pour des méthodes de ce type, il vaut mieux ne pas fermer la session dans ces méthodes.

Et non justement même pas, aucun appel ne fonctionne. Je ne vois pas pourquoi je ne pourrais pas utiliser l'appel à HibernateUtil.closeSession, il faut bien fermer la session à la fin du traitement. Et si tu regarde au début de ces méthode j'ouvre explicitement la session donc normalement elle devrai bien être ouverte.

Ou alors j'ai raté quelque chose ?

Grosso modo, j'ai cette methode (ou autre construite pareil) qui retourne l'exception session is closed :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public User recupererUser(int idUser) { Session s = HibernateUtil.currentSession(); Transaction tx = s.beginTransaction(); User us = (User) s.get(User.class,idUser); tx.commit(); HibernateUtil.closeSession(); return us; }
Alors que c'est la base de l'utilisation d'Hibernate et HibernateUtil, elle marchais parfaitement auparavant mais plus moyen à present ...

20/06/2006, 11h17
K-Kaï

petit up, toujours coincé et je ne vois pas pourquoi je me fais jeté sachant que au moment ou je lance la requète, je suis sure qu'il n'y a qu'un accès en même temps, que c'est l'utilisation de base d'HibernateUtil et que cela fonctionnai auparavant ?
20/06/2006, 13h58
K-Kaï

Et bien aussi mystérieusement que cela s'est arrété de fonctionner, cela remarche. J'ai simplement refait ma classe HibernateUtil, à l'identique et plus aucun problème.

Le mystère reste entier ...
20/06/2006, 15h40
brainstorm

Ayant rencontré un probleme similaire, je vais décrire la solution que l'on y a apporté :

Il y a plusieurs stratégies de chargement de ton objet hibbernate en mémoire (cela sert lorsque tu recupere un objet enorme mais que tu n'est pas sur d'utiliser tout le champs). Parmis celle-ci, tu as le lazy loading qui ne charge que quelques champs de ton objet en memoire et lorsque tu tente d'acceder a un champ non chargé, va rechercher dans la BDD.

Nous fermions aussi la session avec un lazy=true et lorsqu'hibbernate voulais recuperer le champs non chargé a la base, il nous ramenait ton erreur.

Nous sommes passés en lazy = false et plus de probleme. 8-)
20/06/2006, 16h04
K-Kaï

Eheh merci merci mais la non cela ne venait pas de la, j'avais vérifié ceci et je ne tentais pas d'accéder à une collection de mon objet.

Mais par contre je te retourne le conseil, au lieu d'enlever le lazy loading tu pouvais également réattacher ton objet à une session et travailler avec ton objet attaché.
20/06/2006, 16h12
brainstorm

Ben c'est pas trop la philosophie de notre application (n-tiers)

le client de notre couche DAO (souvent la couche service ou business) n'est pas censé travailler sur des objets encore connectés à la BDD (ou je te raconte pas le bronx) donc on ferme systématiquement la session avant de retourner le pojo (on aimerais bien externaliser cela avec un proxy dynamique, mais on ne sait pas le faire :aie: )
22/06/2006, 10h37
K-Kaï

Pffff, blasé voila que je me repren la même exception. Je comprends pas, ca marche puis ca marche plus sans raison apparente.

Vraiment si quelqu'un voit quelque chose qui cloche je suis plus que preneur parceque la ca me bloque completement niveau developpement.
22/06/2006, 18h01
Alwin

Donne le code de HibernateUtils stp

Voici le code de HibernateUtil, c'est simplement celui du tuto Hibernate de dvp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
 
public class HibernateUtil {
	private static final SessionFactory sessionFactory;
 
	 static {
	   try {
	   // Crée la SessionFactory
	   sessionFactory =
		 new Configuration().configure().buildSessionFactory();
	   } catch (HibernateException ex) {
	   throw new RuntimeException("Problème de configuration : "
	   + ex.getMessage(), ex);
	   }
	   }
 
	 public static final ThreadLocal session = new ThreadLocal();
 
	 public static Session currentSession()
			throws HibernateException {
	   Session s = (Session) session.get();
	   // Ouvre une nouvelle Session, si ce Thread n'en a aucune
	   if (s == null) {
	   s = sessionFactory.openSession();
	   session.set(s);
	   }
	   return s;
	   }
 
	 public static void closeSession()
			throws HibernateException {
	   Session s = (Session) session.get();
	   session.set(null);
	   if (s != null)
	   s.close();
	   }
}

23/06/2006, 09h32
cicolas

Pour brainstorm ! Passer en lazy=false surcharge bcp l'application, et le nombre de requêtes ! Tu peux récupérer les dépendances de tes objets via des criteria et fetchMode, ça rend les appels plus propre, et en une requête, tu peuple un objet et toutes ces dépendances.
23/06/2006, 10h24
K-Kaï

Je ne comprends vraiment pas ce qui se passe, quoi que j'essaie de faire sur ma base, il me retourne une exception Session is closed !

Je ne comprends pas ce qui ne va pas.
Je sais pas moi pourquoi session is closed sans arret alors qu'il n'y a aucun accès concurrent pour le moment, que je n'ai qu'une requête qui s'exécute à la fois et ce sont simplement des accès en lecture ....
23/06/2006, 10h50
cicolas

J'imajine que tu l'as fait, mais ajoute des sysout(session==null), pour vérifier si elle est ouverte, et quand elle se ferme...
23/06/2006, 11h21
K-Kaï

Ba oue si tu veux quand j'execute en debug, je suis ma varaible session et elle est dans le bon etat juste avant ma requète sur la base, c'est à dire que le booleen closed et bien a false mais au moment d'executer la requète je mange le session is closed. D'où le fait que je ne sais vraiment plus quoi faire ...

Perso, voici l'implémentation que j'utilise !

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
public class HibernateUtil {
    public static final SessionFactory sessionFactory;
 
    static {
        try {
            // Création de la SessionFactory à partir de hibernate.cfg.xml
            sessionFactory = new Configuration().configure().buildSessionFactory();
        } catch (Throwable ex) {
            // Make sure you log the exception, as it might be swallowed
            System.err.println("Initial SessionFactory creation failed." + ex);
            throw new ExceptionInInitializerError(ex);
        }
    }
 
    public static final ThreadLocal session = new ThreadLocal();
 
    public static SessionFactory getSessionFactory() {
        return sessionFactory;
    }
}

Pour la manipuler :

Code:

Session session = HibernateUtil.getSessionFactory().getCurrentSession();

L'avantage, elle permet l'externalisation du concept de currentSession !

C'est la classe préconisée par Hibernate

26/06/2006, 09h47
K-Kaï

Quelqu'un sait-il comment on s'y prend pour configurer la SessionFactory pour qu'elle fonctionne en ThreadSafe avec les ThreadLocalSessionContext ...
28/06/2006, 01h24
julienito

Je dis peut-être des bétises, mais je crois me souvenir que le fait de faire un
tx.commit() ferme déjà la session, non ?
28/06/2006, 09h27
K-Kaï

Oui mais si un autre thread arrive et demande currentSession alors que le premier est dans le traitement, alors si le tout est mal configuré il aura la même session que le premier, et au moment où le premier thread ferme le session, alors le second l'a dans l'os ....

Je teste actuellement une solution donnée par alwin dans un autre post (mais j'ai un problème dans un fichier de mapping actuellement donc je ne peut pas vraiment voir. Je donnerais des nouvelles plus tard quand j'aurais avancé.

En complément de mon Factory, voici la config de mon hibernate :
Code:

1 2 3  <property name="current_session_context_class">thread</property>

28/06/2006, 10h01
K-Kaï

Je te remercie, tu me l'a deja donné dans un autre post Alwin, je règle mon problème de mapping d'héritage et je revien confirmer ou infirmer le fonctionnement chez moi.
Mais juste comme cela, à quoi te sert le ThreadLocal session dans ton hibernateUtil, tu n'apel pas tres session directement avec factory.getcurrentSession ?

Je ne sais pas ou t'as eu ton HibernateUtil mais mois j'utilise celui ci-dessous et ça fonctionne bien

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
package utils;
 
import org.hibernate.*;
import org.hibernate.cfg.*;
 
public class HibernateUtil {
 
    private static final SessionFactory sessionFactory;
 
    static {
        try {
            // Create the SessionFactory from hibernate.cfg.xml
            sessionFactory = new Configuration().configure().buildSessionFactory();
        } catch (Throwable ex) {
            // Make sure you log the exception, as it might be swallowed
            System.err.println("Initial SessionFactory creation failed." + ex);
            throw new ExceptionInInitializerError(ex);
        }
    }
 
    public static SessionFactory getSessionFactory() {
        return sessionFactory;
    }
 
}

Pour créer la session je fait

Code:

Session session = HibernateUtil.getSessionFactory().getCurrentSession();

Pour la fermeture je fais

Code:

session.close();

Essaie avec ça

28/06/2006, 23h15
julienito

Citation:

Envoyé par K-Kaï

Oui mais si un autre thread arrive et demande currentSession alors que le premier est dans le traitement, alors si le tout est mal configuré il aura la même session que le premier, et au moment où le premier thread ferme le session, alors le second l'a dans l'os ....

Justement, avec le mapping donné par Alwin, la session retournée par getCurrentSession() est spécifique à la thread courante.

La je ne comprend vraiment pas ce qui se passe, j'obtien une exception session is already closed :

Code:

1
2
3
4
5
6
7
8
9
10
 
xException in thread "main" org.hibernate.SessionException: Session was already closed
	at org.hibernate.impl.SessionImpl.close(SessionImpl.java:270)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at org.hibernate.context.ThreadLocalSessionContext$TransactionProtectionWrapper.invoke(ThreadLocalSessionContext.java:301)
	at $Proxy0.close(Unknown Source)
	at persistance.TestPersist.main(TestPersist.java:46)

Le tout sur un simple petit test :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
Session session = HibernateUtil.getSessionFactory().getCurrentSession();
		Transaction tx1 = session.beginTransaction();
 
		User us = new User();
		us.setNom("Martin");
		us.setPrenom("Philippe");
 
		us.setAdresse("18, rue des cerisiers");
		us.setCp(63000);
		us.setVille("Clermont Ferrand");
		us.setTel("0473658941");
		us.setMailPerso("lekaille@gmail.com");
 
		auth.setLogin("toto");
		auth.setPass("tintin");
 
		session.persist(auth);
 
		us.setAuthentification(auth);
 
		session.persist(us);
		tx1.commit();
		session.close();

Avec la classe HibernateUtil de Alwin ainsi que la même ligne dans le fichier de configuration.

...

03/07/2006, 09h22
K-Kaï

Bon alors après test, le problème vient du

Code:

HibernateUtil.getSessionFactory.getCurrentSession()

Avec

Code:

HibernateUtil.getSessionFactory.openSession()

Cela marche ...

Sauf que moi j'ai besoin du getCurrentSession(), voyez vous ce qui peut ne pas aller et ou ?

Plus précisément, avec le getCurrentSession() tout se passe bien jusqu'à l'appel à session.close() ? Es ce que avec le getCurrentSession et Hibernate configuré pour fonctionner en Thread Safe (façon Alwin) le close() devient inutile ?

Bon j'avais le même pb et avec ce code pour creer une session, ça marche nickel
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public static Session currentSession() throws HibernateException { Session s = (Session) session.get(); if (s != null && ! s.isConnected()) //on check l'etat de la Session precedente s = null; // Ouvre une nouvelle Session, si ce Thread n'en a aucune if (s == null) { s = sessionFactory.openSession(); session.set(s); } return s; }
Pour ce qui est de ta question sur le getCurrentSession, en faisant un tour sur la javadoc, ils disent ceci :

Citation:

public Session getCurrentSession()
throws HibernateException

Obtains the current session. The definition of what exactly "current" means controlled by the CurrentSessionContext impl configured for use.

Note that for backwards compatibility, if a CurrentSessionContext is not configured but a JTA TransactionManagerLookup is configured this will default to the JTASessionContext impl.

Entre temps j'ai repéré d'ou venait l'exception, la session est fermé au moment du commit (auto close session il me semble) donc voila pourquoi je me faisais jeté à chaque fois que j'essayais de fermer ma session explicitement.

A titre indicatif, il semblerait qu'il faille procéder ainsi (toujours dans le contexte multithread):
Code:

1 2 3 4 5 6 7 8 9 10 11 12 try { Session s = HibernateUtil.getSessionFactory().getCurrentSession(); //Traitements //commit } catch(HibernateException e){ //rollback } finally{ HibernateUtil.getSessionFactory().getCurrentSession().close(); }

Il ne me semble pas que le commit ferme la session, en tous cas, ce n'est pas cela qui génère l'erreur (que j'ai aussi).

Pour ma part, j'ai analysé mon HibernateUtil, en particulier la méthode currentSession() et je me rends compte qu'il n'ouvre une nouvelle session que si celle-ci est null. Mais je suppose qu'un session peut-être fermée et ne pas être null (à moins que je n'y connais rien :?).

En tous cas, j'ai ajouté un test "!s.isOpen()" et ça fonctionne :
Code:

1 2 3 4 5 6 7 8 9 10 public static Session currentSession() throws HibernateException { Session s = (Session) session.get(); // Ouvre une nouvelle Session, si ce Thread n'en a aucune if (s == null || !s.isOpen()) { s = sessionFactory.openSession(); session.set(s); } return s; }
De cette façon il m'ouvre une session, qu'elle soit null ou fermée.

J'espère que cela pourra aider quelqu'un ;)