Reprise du message précédent :
question conne hein, mais ton query broker c'est un singleton qui utilise une connection donnée, deux clients attaquent ton site web, les deux threads utilisent ton query broker, le premier close ta connection, qu'arrive t'il au second ?
 
c'est moi qui a mal suivi ou y a une grosse coucouille là ? ce serait mieux si c'était pas un singleton et que tu avais bien une connection par thread nan ? paske la bon...ton pool de connection il a la vie un peu tranquile j'ai l'impression
 
(quelque chose me dit que je viens de dire une connerie et que plusieurs post m'ont échappé nan ? )

j'ai pas tout lu mais j'ai vu une grosse erreur !!!
 
Ton object QueryBrocker n'est pas du tout threadsafe dans l'état, et ta servlet va être accéder de manière concurente (plusieurs thread vont executer en même temps doPost()) => tu vas avoir de gros gros problèmes si tu mets ca en prod !!!

ah bah j'ai pas du dire de connerie alors \o/

effectivement je suis grilled, quoi

quand je vous dis que les singleton, ça pue du cul
 
elle pourrait faire un ThreadLocal

Pour l'instant son problème est plutot de gérer son probleme de connexion qui ne retourne pas ds le pool.
Chaque chose en son temps
MAis c'est vrai que le singleton implémenté ici n'est pas threadsafe.

y a plus simple pour faire du threadsafe ...

le plus simple serait de virer le singleton et de stocker/instancier son querybrocker dans une variable locale a la methode service()

bof. Il faudrait refaire la recherche du datasource à chaque appel.
 
Le mieux serait de faire une factory de querybrocker => la recherche JNDI est faite dans la factory une fois pour toute.

 
En fait je parlais plus du probleme d'ouverture/fermeture de connexion car c'est lui qui pose vraiment probleme ici.  
Le singleton devrait plutot porter sur le QueryBrokerFactory et non sur le QueryBroker
 
D'ailleurs hop, un petit lien : http://www.javaworld.com/javaworld [...] dsafe.html
 
Quand à son problème de connexion qui retourne pas dans le pool...

sans blague

merci à tous pour vos réponses. je ne suis pas partie en vacances et je n'ai pas abandonné mon problème non.
 
je ne l'ai pas résolu non plus malheureusement
mais c'est normal puisque gt à l'école et non au boulot
 
bref j'ai parlé de mon pb avec un prof et il me dit que je dois forcément mal fermer mes connections à un moment. il a aussi tilté comme vous sur mon queryBroker qui n'est pas thread safe et en plus mal utilisé dans mes servlets
 
bon je crois que je vais devoir revoir complètement maz structure à ce sujet. dites moi comment vous faite vous pour vos appli web ?
vous faites aussi une classe qui s'occupe de gérer les connexions ? comment vous l'utilisez tout ça quoi !
en gros j'ai besoin de vous pour me sortir la m.... de mes yeux !

bon je pense avoir ENFIN résolu mon pb !  
 
Alors j'ai remis mes instances du queryBroker dans les méthodes de mes classes métiers et j'ai modifié mon QueryBroker.getInstance qui renvoyait l'instance existante de celui-ci et je crois bien que ct ça qui foutait le bordel. Du coup à chaque fois il créé une nouvelle instance.
 
j'ai fais pas mal de refresh et je n'ai pas rencontré mon fameux problème.
 
maintenant je vais le rendre thread safe puisque ce n'est pas fait et que c'est clairement MAL !  
bon j'me souviens plus trop de ça mais je m'en vais à la recherche d'un bon tuto
 
En tout cas merci beaucoup pour vos remarques et critiques qui ont été constructives et m'ont fait avancer
j'aimerai par la suite vous demander à nouveau votre avis sur des bouts de code pour voir si ce que j'ai fait n'est pas trop crade