Pb de connexions "Pool exhausted"

Version imprimable

Bonjour,

j'ai un pb sur un de mes projets java/web.
Au bout d'un certain temps d'utilisation, l'appli plante, et dans les log, j'ai :

Code:

1
2
org.apache.commons.dbcp.SQLNestedException: Cannot get a connection, pool exhausted, cause: 
java.util.NoSuchElementException: Timeout waiting for idle object

Et ca, j'arrive pas a le comprendre car toutes les connexion sont systematiquement fermes .
Ex :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
sqlQuery = "select * from ??? where ?? = ???";
		try{
			conn = new Connexion(applicationRessource).getConnexion();
			prst = conn.prepareStatement(sqlQuery);
			ResultSet rs = prst.executeQuery();
 
			if (rs.next()){
				...
			}
			if (prst != null) try {prst.close();prst = null;} catch (SQLException e) {}
			if (conn != null) try {conn.close();conn = null;} catch (SQLException e) {}
			if (rs != null) try {rs.close();rs = null;} catch (SQLException e) {}
			return total;
		} catch (SQLException e1) {
			...		}

Est ce que vous pouvez me dire si c'est la bonne methode ou pas.

D'avance, merci

08/04/2005, 15h18
christopheJ
Bonjour,

C'est ce qu'on appelle une fuite de connexions. Tu ne dois pas libérer une connexion a un endroit...
Une piste est dans la structure de ton code : la fermeture des connections doit se faire dans un bloc finally.
Dans ton code si l'exécution de la requete lève une exception, tu sautes directement dans le bloc catch et ne libère jamais ta connexion.
la structure est :
Code:

1 2 3 4 5 6 7 8 9 try{ recupération connexion execution requete traitement réponse }catch(SQLException ex){ traitement exception }finally{ fermeture Resultset, Statement, Connection }
Sinon tu as aussi des options dans les parametres du pools de connection DBCP pour forcé la récupération de connexions inactives. J'en parle dans mon tutoriel sur les pools de connections dans Tomcat :
http://christophej.developpez.com/tu...ee/pooltomcat/

L'accès standard se fait de la manière suivante :

Code:

1
2
3
4
5
6
Context ctx = new InitialContext();
Datasource ds = (Datasource)ctx.lookup("java:comp/env/jdbc/MyDatasource");
Connection cnx = ds.getConnection();
 
...
...

Dans ton web.xml, tu rescence un datasource avec un <resource-ref>

Code:

1
2
3
4
5
 
<resource-ref>
  <res-ref-name>jdbc/MyDatasource</res-ref-name>
  <res-type>javax.sql.DataSource</res-type>
</resource-ref>

et dans ton serveur.xml tu défini ton pool !

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
 
<Context...>
  <Resource name="jdbc/MyDatasource" type="javax.sql.DataSource"/>
  <ResourceParams name="jdbc/MyDatasource">
 
    <parameter>
      <name>user</name>
      <value>dbusername</value>
    </parameter>
 
    <parameter>
      <name>password</name>
      <value>dbpassword</value>
    </parameter>
 
    <parameter>
      <name>driverClassName</name>
      <value>org.hsql.jdbcDriver</value>
    </parameter>
 
   <parameter>
      <name>driverName</name>
      <value>jdbc:HypersonicSQL:database</value>
    </parameter>
 
  </ResourceParams>
 
  ...
 
</Context>

le cnx.close() final ne fermeras pas la connexion mais la replaceras dans le pool.[/code]

08/04/2005, 20h59
Sherkhan

Bah c'est exactement ce que je fais dans mon code.
En fait, la partie

Code:

conn = new Connexion(applicationRessource).getConnexion();

appelle une de mes classes qui se charge justement d'aller recuperer le datasource etc ...

J'ai verifie, je ferme bien toutes mes connexions. Ou en tout cas, j'en ai bien l'impression ...
09/04/2005, 08h40
Alwin

Méa culpa alors, je ne comprennais pas pourquoi tu passais un paramètre "applicationResource" à ta méthode.

sinon, tu ne ferme pas toutes les connexions dans ton code dans le cas d'une erreur SQLException

Ajoute le bloc "finnaly" comme te l'indique christopheJ.

Autre chose: tu utilise quoi comme base de données et qu'elle est ta requête réelle ?

N'utiliserais tu pas du Oracle avec des requetes genre :

"select * /* FIRST_ROWS */ from maTable where...... order by ....." ?
09/04/2005, 09h54
Sherkhan

En fait, vous avez raison et je suis unr grosse truffe.
Je pensais avoir mis les try/catch aux bons endroits mais en fait pas du tout !
Dans le cas d'une erreur SQL, la connexion, effectivement, ne se ferme pas ...

Merci de votre aide !