Discussion :

[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é.

[Alwin]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
<!-- Enable Hibernate's automatic session context management -->
<property name="current_session_context_class">thread</property>

[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 ?

[ouedmouss]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Pour la fermeture je fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

session.close();

Essaie avec ça

[julienito]

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.

[K-Kaï]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

...

[K-Kaï]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

HibernateUtil.getSessionFactory.getCurrentSession()

Avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[drixfkc]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

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.

[K-Kaï]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();
}

[DaveRidic]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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