Retour Hibernate aléatoire

**_Babar_** · 02/08/2010, 18h24

Bonjour

je suis dans le développement d'un site JavaEE avec MySql, et Hibernate (3) réagit bizarrement :
Lorsque je multiplie les appels au Dao (toujours de la même manière, en boucle, pour incrémenter un total par exemple) il arrive (1 fois sur 10) que les objets ne soient pas récupérés, ou bien dans un état précédent(avec un total faussé donc).

Le problème semblerai venir d'un cache hibernate.
J'ai ajouté des session.flush() mais ca n'a pas aidé.
Je me demande si ca peut venir de mon hibernateUtil, ou bien de mon mapping peut etre ?...

Tous vos commentaires / liens sont les bienvenus.
Merci !

HibernateUtile :

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
26
27
28
29
30
31
32
33
34
35
36
37
 
 
	private static Log					log					= LogFactory.getLog(HibernateUtile.class);
 
	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 getSession()
	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();
	}

Mon dao

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
 
Compte compte = null;
 
		try {
 
			Session session = HibernateUtile.getSession();
 
			Query query = session.createQuery("from Compte where profil=:profil and partenaire.code=:code");
 
			query.setString("profil", profil);
			query.setString("code", codePartenaire);
 
			compte = (Compte) query.uniqueResult();
 
.....

ma config hibernate

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
 
<?xml version='1.0' encoding='utf-8'?>
<!DOCTYPE hibernate-configuration PUBLIC
"-//Hibernate/Hibernate Configuration DTD//EN"
"http://hibernate.sourceforge.net/hibernate-configuration-3.0.dtd">
 
<hibernate-configuration>
	<session-factory>
		<property name="hibernate.connection.driver_class">com.mysql.jdbc.Driver</property>
		<property name="hibernate.connection.url">jdbc:mysql://localhost:3306/mydb</property>
		<property name="hibernate.connection.username">root</property>
		<property name="hibernate.connection.password">root</property>
		<property name="show_sql">false</property>
		<property name="dialect">org.hibernate.dialect.MySQLDialect</property>
 
        <property name="current_session_context_class">thread</property>
        <property name="hibernate.current_session_context_class">org.hibernate.context.ThreadLocalSessionContext</property>
 
		<!-- Mapping files -->
        <mapping resource="fr/globalaction/hibernate/Action.hbm.xml" />
......
	</session-factory>
</hibernate-configuration>

un exemple de Dao

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
	Session session = HibernateUtile.getSession();
 
	Query query = session.createQuery("from Compte where profil=:profil and partenaire.code=:code");
 
	query.setString("profil", profil);
	query.setString("code", codePartenaire);
 
	compte = (Compte) query.uniqueResult();

**juridakus** · 04/08/2010, 15h07

Essaies d'ajouter ces propriétés dans ton fichier hibernate.cfg.xml

Code :

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

1
2
3
 
<property name="hibernate.cache.use_query_cache">false</property>
<property name="hibernate.cache.use_second_level_cache">false</property>

**_Babar_** · 05/08/2010, 10h51

Merci de ta réponse

les deux lignes que tu m'as précisé correspondent apparemment au cache de second niveau. J'ai lu qu'il n'était pas actif par défaut, mais j'essaye quand même dès que j'arrive a redémarrer mon tomcat

a cause d'une erreur filterStart ...

**juridakus** · 05/08/2010, 11h42

La première ligne concerne le query_cache utilisé prioritairement par Hibernate pour les requête de type session.createQuery("..") comme tu le fais dans ton Dao.

**_Babar_** · 05/08/2010, 12h11

Et ca ne résout pas mon problème ...

Merci quand même ! Je cherche toujours ..

(et pour le filterStart, Struts 2 a besoin de freemarker.jar mais c'est une autre historie .. )

**_Babar_** · 10/08/2010, 11h44

Est ce qu'il pourrait y avoir un lien avec mes hashCode et equals ?
(Puisqu'ils peuvent générer des erreurs "existing object with same identifier.." )

Lorsque je les génère, je n'inclue pas les relations a 1-n, n-1 et n-n.

**tchize_** · 10/08/2010, 11h54

tu stocke tes session dans un threadlocal. Tu n'oublierais pas de faire des session.close() après la requete utilisateur par hasard? Losqu'une session a chargé un objet, elle n'ira plus jamais sur la db recharger cet objet (à mois de passer par session.evict ou session.merge). Donc si tu oublie le close et que tu le laisse dans le threadlocal, il est possible qu'une requete http suivante réutilise cette session, et donc récupère l'objket dans l'état où la session le connait.

Enfin, si tu as des requetes en //, n'oublie pas que tu dois les isoler l'une de l'autre pour ne pas corromptre ta base de donnée. Pour ça, l'utilisation de transaction et d'un niveau d'isolation suffisant de celle-ci (config au niveau du datasource ou de hibernate)

**_Babar_** · 10/08/2010, 16h36

Merci de ta réponse !

mes sessions sont bien fermés.

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
26
27
28
29
30
 
public List<Total> getTotauxParPartenaireIdEtMois(int partenaireId, String mois) throws GaException {
 
		log.debug("getTotauxParPartenaireIdEtMois - " + partenaireId + "/" + mois);
		try {
 
			Session session = HibernateUtile.getSession();
			Query query = session.createQuery("from Total where periode=:mois and partenaireId=:partenaireId");
			query.setInteger("partenaireId", partenaireId);
			query.setString("mois", mois);
 
			List<Total> result = new ArrayList<Total>();
 
			result.addAll(query.list());
 
			for (Iterator<Total> iterator = result.iterator(); iterator.hasNext();) {
				Total total = iterator.next();
				log.info("total " + total.getType() + " " + total.getTotal());
			}
 
			return result;
 
		} catch (RuntimeException e) {
			log.error("Error - " + e.getMessage());
			e.printStackTrace();
			throw e;
		} finally {
			HibernateUtile.closeSession();
		}
	}

et mon HibernateUtile :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
 
 
public class HibernateUtile {
 
	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> session = new ThreadLocal<Session>();
 
	public static Session getSession()
	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();
	}
	}

... qu'appelles-tu des requetes // ?

Merci

**tchize_** · 10/08/2010, 16h50

eds requetes qui se font en même temps. Ce que tu dois d'office prévoir en environnement web, puisque ça surviens dès que deux utilisateur font une opération en même temps.

**_Babar_** · 10/08/2010, 18h13

ok merci

c'est ptet ca le problème.
Je n'utilise la transaction que pour les save, update, saveOrUpdate. J'essayerai de l'ajouter pour les select.

Pour finir, est ce que ma transaction est bien utilisée comme ca : ?
(HibernateUtile et configuration hibernate sont dans les posts précédents)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
try {
			Session session = HibernateUtile.getSession();
			Transaction transaction = session.beginTransaction();
			session.saveOrUpdate(applet);
			transaction.commit();
		} catch (RuntimeException e) {
			log.error("Error - " + e.getMessage());
			e.printStackTrace();
			throw e;
		} finally {
			HibernateUtile.closeSession();
		}

Merci pour tout !

**tchize_** · 10/08/2010, 19h33

oula: il sort d'ou le applet, bien de la même session?. Tu ne dois jamais réutiliser un objet détaché d'une session, il ne sont pas mis à jour en mémoire par hibernate! Sinon, imagine ceci:

Session
prendre objet X
fin de session

**autre thread**
session
prendre objet X, le modifier, le sauver
fin de session
** fin autre thread, retour premier thread**
Session
sauver X
fin de session.

**_Babar_** · 11/08/2010, 11h42

j'ai bien imaginé ce cas la. A vrai dire, je me suis basé sur les sources que j'ai récupéré de mon dernier client, un gros webmarchand.
Le grosse différence est la présence d'ejb pour accéder aux Dao. Mais je ne pense pas que ca change quelquechose.

Le test de ce cas, je l'ai fait avec succès sur une seule machine, en utilisant firefix en parallèle d'IE.

Si tu as une bonne doc hibernate la dessus ca m'intéresse. Sinon je vais me plonger dans la doc officielle.

Merci encore

**_Babar_** · 11/08/2010, 14h16

re !

J'ai mis tout mon traitement (récupération de l'objet puis update) dans le Dao, et le problème persiste.

J'ai mieux identifier le problème : entre deux requetes, je recharge la page de mon navigateur pour effectuer ma simulation, et c'est ca qui perturbe mes données. Mais je ne comprends pas vraiment pourquoi.

Merci encore de votre aide a tous.

**tchize_** · 11/08/2010, 14h24

faut êter plus précis sur ce qui foire. Tu garde tes objet hibernante en session HTTP? C'est à éiter autant que possible.

**_Babar_** · 12/08/2010, 09h34

Effectivement, je ne comptais pas m'arreter la.

Tu veux dire qu'il vaut mieux passer par des DTO dès qu'on sort de la couche hibernate ?

En l'occurrence, mon dao qui foire recoit juste un montant, qu'il ajoute a un total. Donc le dao récupère l'objet Total, incrémente une valeur, puis l'update.

Lorsque je rafraichis ma page pour rappeler ce même dao, le Total n'est pas toujours

un petit exemple de mes logs du dao, j'incrémente les totaux de 1 a chaque fois :

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
 
2010-08-11 13:54:14 INFO  [PartenaireDao] Nouveaux totaux : 1.00
2010-08-11 13:54:14 INFO  [PartenaireDao] Nouveaux totaux : 1.00
2010-08-11 13:54:19 INFO  [PartenaireDao] enregistrerDon
2010-08-11 13:54:19 INFO  [PartenaireDao] Totaux incrémentés : 2.00
2010-08-11 13:54:19 INFO  [PartenaireDao] Totaux incrémentés : 2.00
2010-08-11 13:54:26 INFO  [PartenaireDao] enregistrerDon
2010-08-11 13:54:26 INFO  [PartenaireDao] Totaux incrémentés : 3.00
2010-08-11 13:54:26 INFO  [PartenaireDao] Totaux incrémentés : 3.00
2010-08-11 13:54:33 INFO  [PartenaireDao] enregistrerDon
2010-08-11 13:54:33 INFO  [PartenaireDao] Totaux incrémentés : 4.00
2010-08-11 13:54:33 INFO  [PartenaireDao] Totaux incrémentés : 4.00
2010-08-11 13:54:36 INFO  [PartenaireDao] enregistrerDon
2010-08-11 13:54:36 INFO  [PartenaireDao] Totaux incrémentés : 3.00
2010-08-11 13:54:36 INFO  [PartenaireDao] Totaux incrémentés : 3.00

**tchize_** · 12/08/2010, 10h51

la "récupération, incrémentation, update" se fait bien dans une seule transaction hibernate? Avec un isolation de type SERIALIZABLE au niveau JDBC? Il pourrait être utilie dans ton log que tu affiche aussi le thread concerné et le hashcode d'origine de l'objet (System.identityHashcode(objet)) que tu update + de la session, on y verrais plus clair sur qui fait quoi

**_Babar_** · 12/08/2010, 15h00

la "récupération, incrémentation, update" se fait bien dans une seule transaction hibernate?

sir, yes, sir !

Avec un isolation de type SERIALIZABLE au niveau JDBC?

Mes objets mappés implémentent Serializable, oui

Il pourrait être utilie dans ton log que tu affiche aussi le thread concerné et le hashcode d'origine de l'objet (System.identityHashcode(objet)) que tu update + de la session, on y verrais plus clair sur qui fait quoi

Je ne passe pas explicitement par un thread, j'accède uniquement a la session a laquelle j'associe une transaction :

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
26
27
 
		try {
			Session session = HibernateUtile.getSession();
			Transaction transaction = session.beginTransaction();
 
			Query query = session.createQuery("...");
...
			List<Total> totauxPartenaire = query.list(); 
 
				for (Iterator<Total> iterator = totauxPartenaire.iterator(); iterator.hasNext();) {
					Total total = iterator.next();
...
					session.update(total);
 
...
				}
			}
			transaction.commit();
 
		} catch (RuntimeException e) {
			log.error("Error - " + e.getMessage());
			e.printStackTrace();
			GaException gae = new GaException(e);
			throw gae;
		} finally {
			HibernateUtile.closeSession();
		}

System.identityHashcode(objet) je ne connais pas, je cherche comment l'utiliser

"Que j'update plus de la session" ? qu'est ce que tu veux dire par la ?

Merci de ton aide encore une fois

**tchize_** · 12/08/2010, 16h53

Envoyé par _Babar_

Mes objets mappés implémentent Serializable, oui

Rien à voir, je demande ton isolation jdbc, ce qui permettra de déterminer si oui ou non tu a autorisé les dirty read, les phantom read, etc. Bref tout ce qui peux corrompre une DB

Je ne passe pas explicitement par un thread, j'accède uniquement a la session a laquelle j'associe une transaction :

non mais ton conteneur associe un thread à chaque connexion http entrante (d'ou l'utilisation de ton threadlocal), il serait bon d'afficher le nom du thread histoire qu'on sache qui fait quoi! Affiche aussi les requete hibernate (show_sql = true) pur qu'on voie ce que fait hibernate pendant ce temps.

System.identityHashcode(objet) je ne connais pas, je cherche comment l'utiliser

ca renvoie un int, t'as qu'à l'afficher

"Que j'update plus de la session" ?

Le hashcode du bean que tu update + le hashcode de la session, qu'on voie les deux
Le premier permettra de voir si oui ou non on recycle un bean, le deuxième si oui ou non on recycle une session...

**_Babar_** · 12/08/2010, 18h23

ok ! par défaut l'isolation jdbc était REPEATABLE READ, je l'ai configurée en SERIALIZABLE, et j'obtiens des erreurs dès la première requete, après un timeout d'environ 2 minutes :

Code :

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

1
2
3
4
5
6
7
8
9
 
Could not execute batch update
...
Caused by: java.sql.BatchUpdateException: Lock wait timeout exceeded; try restarting transaction
        at com.mysql.jdbc.PreparedStatement.executeBatchSerially(PreparedStatement.java:2018)
        at com.mysql.jdbc.PreparedStatement.executeBatch(PreparedStatement.java:1454)
        at org.hibernate.jdbc.BatchingBatcher.doExecuteBatch(BatchingBatcher.java:70)
        at org.hibernate.jdbc.AbstractBatcher.executeBatch(AbstractBatcher.java:268)
        ... 55 more

mais je comprends pas :/
Selon mes recherches ca correspond a un problème de plusieurs transactions qui s'emmelent, pourtant je pense n'utiliser qu'une transaction :

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
26
27
 
		try {
			Session session = HibernateUtile.getSession();
			Transaction transaction = session.beginTransaction();
 
			Query query = session.createQuery("...");
...
			List<Total> totauxPartenaire = query.list(); 
 
				for (Iterator<Total> iterator = totauxPartenaire.iterator(); iterator.hasNext();) {
					Total total = iterator.next();
...
					session.update(total);
 
...
				}
			}
			transaction.commit();
 
		} catch (RuntimeException e) {
			log.error("Error - " + e.getMessage());
			e.printStackTrace();
			GaException gae = new GaException(e);
			throw gae;
		} finally {
			HibernateUtile.closeSession();
		}

Est ce que c'est parce que je fais un select de plusieurs lignes ?

**tchize_** · 12/08/2010, 19h56

t'as une transaction qui te bloque, probablement lancée par une autre méthode et / ou un autre session. Dans le code que tu présente tu n'a qu'une seule transaction. L'erreur que tu sort ne fait que démontrer ce qui se soupconne depuis le début, t'as des modifications concurrente entre plusieurs sessions hibernate. C'est probablement un transaction pas terminée lors d'une autre requete qui pose problème.

Retour Hibernate aléatoire

Hibernate Java

Discussions similaires

Partager

Partager