[tomcat] freeze ...

**Valerio** · 11/04/2005, 22h05

Bonsoir,

j'ai un petit problème avec Apache Tomcat 5.0.28
J'ai une page qui fonctionne correctement (avec des requêtes SQL) ... si je fais 2 ou 3 refresh de la page dans le navigateur, tomcat a l'air de freezer ... obliger de le stopper et redémarrer.

Quelqu'un aurait un programme pour "surveillez" ou analyser la charge de travail du serveur ?

**ZeKiD** · 11/04/2005, 22h42

Probablement des connexions mal libérées.
Tu as pensé à regarder, ensuite Tomcat essaie de connecter mais il n'y arrive plus.
C'est probablement quelque chose de ce genre.
A vérifier.

**Valerio** · 12/04/2005, 08h55

Alors je pense que mes connexions sont libérées ...
voici le genre de requete qui se trouve dans la page index.jsp :

Code :

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

1
2
3
4
5
6
7
8
 
<%
float montant_esth_produits = RequetesSQL.esth_montantProduits(null);
if (montant_esth_produits == -1)
  out.print("<u>pas de ventes</u>");
else
  out.print(montant_esth_produits+" &euro;");
%>

j'ai en fait 4 requête du meme style sur la page.

et la classe RequetesSQL :

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
 
	public static float esth_montantProduits(Date date){
	    float montant = -1;
 
        // Récupère la connexion au DataSource
        Connexion conn = new Connexion();
        conn.init();
 
        PreparedStatement pstmt=null;
        ResultSet rs=null;
        try {
            if (date==null){
                Calendar cal = Calendar.getInstance();
                date = new Date(cal.getTimeInMillis());
            }
       		pstmt = conn.getConn().prepareStatement("SELECT total_produits FROM esth_livre_caisse WHERE date_debut=?;");
       		pstmt.setDate(1, date);
            rs = pstmt.executeQuery();
 
            if(rs.first()){
                montant = rs.getFloat("total_produits");
            }
            conn.getConn().close();
        }
        catch(SQLException sqle){
            //TODO voir si on voit le message
            System.err.println(sqle.getMessage());
        }
        conn=null;
        return montant;
	}//fin esth_montantProduitsEnCours

et ma classe Connexion :

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
package be.cityform.administration.sql;
 
import java.sql.Connection;
import java.sql.SQLException;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.sql.DataSource;
 
public class Connexion {
 
    /** Etat de la connexion */
    private boolean connected = false;
    /** La référence au DataSource */
    private DataSource ds = null;
 
    /**
     * Initialise la connexion au DataSource
     */
    public void init() {
        try {
            Context initContext = new InitialContext();
            Context envContext = (Context) initContext.lookup("java:/comp/env");
            ds = (DataSource) envContext.lookup("jdbc/postgres");
 
            if (ds != null) {
                Connection conn = ds.getConnection();
                if (conn != null) {
                    connected = true;
                    conn.close();
                    connected = false;
                }
            }
        } catch (Exception e) {
            e.printStackTrace();
        }
    }//fin init
 
    /**
     * Retourne la session de connexion
     * 
     * @return Connection
     * @throws SQLException
     */
    public Connection getConn() throws SQLException {
        return ds.getConnection();
    }//fin getConn
 
    /**
     * Permet de savoir si on est connecté à la base de données
     * 
     * @return boolean
     */
    public boolean isConnected() {
        return connected;
    }//fin isConnected
 
}//fin Connexion

Je sais que dans un premier temps c'est pas super bien codé, mais il faut livrer une version qui fonctionne, j'optimiserai par la suite ...

**ZeKiD** · 12/04/2005, 09h20

Il vaut mieux faire ton close de ta connection dans ton finally de ton try catch.
Car là en cas d'erreur SQL, tu ne passes pas dans ton close.
A voir.

**Valerio** · 12/04/2005, 09h25

Ca donnerait un truc du genre :

Code :

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

1
2
3
4
5
6
7
8
 
        finally{
            try{
                conn.getConn().close();
            }
            catch(SQLException sqle){}
            conn=null;
        }

C'est une bonne méthode d'ouvrir et fermer la connexion au DS a chaque requete SQL ou il vaut mieux ouvrir une seule fois la connexion au début ?

**ZeKiD** · 12/04/2005, 09h27

Oui tout à fait.

**Valerio** · 12/04/2005, 09h44

Toujours le meme probleme

Je peux rafraichir 4 fois ... avec intervalle de 2 secondes .... ensuite il n'affiche plus rien ... il ne sait plus se connecter au module Web en fait ...
mais tomcat fonctionne tjs si je vais sur la page d'accueil http://localhost:8080/ il l'affiche sans prob.
Donc en gros c'est pas Tomcat qui freeze, mais mon application

Je vois pas comment je pourrais faire pour voir d'ou pourrait venir le problème ...

**Valerio** · 12/04/2005, 09h46

J'ai mis le serveur en mode debug mais il n'affiche rien comme message, et rien dans les logs non plus ...

**ZeKiD** · 12/04/2005, 10h10

Tu n'as rien dans les logs ?
C'est étonnant çà.
As tu essayé de mettre des traces. System.out, partout dans ta JSP.
Comme çà tu pourras vérifier l'ensemble des traces et savoir où plante Tomcat.

**Valerio** · 12/04/2005, 10h47

Il plante, par exemple, dès que j'utilise 2 fois une requete SQL :

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
 
         <%
		 float montant_esth_produits = RequetesSQL.esth_montantProduits(null);
		 if (montant_esth_produits == -1)
		 	out.print("<u>pas de ventes</u>");
		 else
		 	out.print(montant_esth_produits+" &euro;");
         %>
et
         <%
		 float montant_esth_soins = RequetesSQL.esth_montantSoins(null);
 		 if (montant_esth_soins == -1)
		 	out.print("<u>pas de ventes</u>");
		 else
		 	out.print(montant_esth_soins+" &euro;");
         %>

les 2 methodes (esth_montantSoins et esth_montantProduits) sont identiques si ce n'est la requete SQL qui change bien évidement (oui je vous ai dit que c'était codé a la va vite

)

pourtant je détruit bien la connexion a chaque fois ...

Je devrais peut etre utiliser autre chose pour gérer mes connexion à la bdd ? (qui est PostGreSQL au passage)
Genre Hibernate ou Spring ...

**ZeKiD** · 12/04/2005, 11h10

Ok je crois savoir.
Tu as oublié de fermer ton ResultSet !!
A celle là tu n'y avait pas pensé !!
Ah oui! Il faut explicitement le fermer.
Désolé.

**Valerio** · 12/04/2005, 11h21

Après avoir fermé les RS et PSTMT , redéployer toute l'application , ... le même probleme ce produit ... ya vraiment quelque chose qui ne tourne pas rond.
Je vais essayer de passer l'application sur un serveur 5.5.9 ...
mais pour le moment j'ai des erreurs SQL du genre :
Cannot create JDBC driver of class '' for connect URL 'null'

Je vais voir en utilisant Hibernate ....

**Valerio** · 12/04/2005, 11h49

Bon, pour finir je suis resté avec Tomcat 5.0.28 ... tjs le meme probleme, au bout d'un certain nombre de refresh ... il "freeze"

j'ai changé le driver JDBC pour PostGresql.
J'utilisais la version 7.3 du driver pour acceder un postgresql 8.0 (peut etre la cause de probleme).
alors j'essaye d'utiliser la version 8.0 du driver (celle qui correspond pour postgresql 8.0)

et bien voila autre chose, j'ai des erreurs du type :

Code :

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

1
2
 
Operation requires a scrollable ResultSet, but this ResultSet is FORWARD_ONLY.

qui s'affiche dans la console Tomcat.

ensuite lorsqu'une requete essaye de s'executer j'arrive sur la page d'erreur :

Code :

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

1
2
3
4
5
6
 
type :  Status report
 
message : [SQLException] Operation requires a scrollable ResultSet, but this ResultSet is FORWARD_ONLY.
 
description : The server encountered an internal error ([SQLException] Operation requires a scrollable ResultSet, but this ResultSet is FORWARD_ONLY.) that prevented it from fulfilling this request.

Je suis un peu perdu la

PS : voici mon fichier de contexte dans : /META-INF/context.xml

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
39
40
41
42
43
 
<Context path="/administration" docBase="C:\Java\jakarta-tomcat-5.0.28\webapps\administration" debug="1" reloadable="true">
<Logger className="org.apache.catalina.logger.FileLogger" prefix="localhost_administration_log" suffix=".txt" timestamp="true"/>
 
   <Resource name="jdbc/postgres" auth="Container"
              type="javax.sql.DataSource"/>
 
<ResourceParams name="jdbc/postgres">
  <parameter>
    <name>factory</name>
    <value>org.apache.commons.dbcp.BasicDataSourceFactory</value>
  </parameter>
  <parameter>
    <name>driverClassName</name>
    <value>org.postgresql.Driver</value>
  </parameter>
  <parameter>
    <name>url</name>
    <value>jdbc:postgresql://localhost:5432/cityform</value>
  </parameter>
  <parameter>
    <name>username</name>
    <value>postgres</value>
  </parameter>
  <parameter>
    <name>password</name>
    <value>*******</value>
  </parameter>
  <parameter>
    <name>maxActive</name>
    <value>20</value>
  </parameter>
  <parameter>
    <name>maxIdle</name>
    <value>10</value>
  </parameter>
  <parameter>
    <name>maxWait</name>
    <value>-1</value>
  </parameter>
</ResourceParams>
 
</Context>

**Alwin** · 12/04/2005, 11h53

la méthode rs.first() doit être remplacée par un

Code :

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

1
2
3
4
5
6
 
if( rs.next()) {
   ...
} else {
  // pas de donnée retournée
}

first(), last() et autres méthodes de navigations nécessite d'avoir un resultset navigable, ce qui n'est pas le cas avec un statement standard !

**Valerio** · 12/04/2005, 12h01

ok j'ai changé mes requetes SQL avec la methode next().
Ca fonctionne, merci

mais j'ai tjs ce probleme de freeze apres un refresh de la page principale maintenant ...

Vous croyez que ca vient de la base de données ?

**ZeKiD** · 12/04/2005, 12h12

Bon bah on a fini par y arriver !!!

**Alwin** · 12/04/2005, 12h25

Bon, déjà, quand tu fais un new Connexion() tu ouvre une connexion que tu ferme sur le champ, peut tu nous expliquer pourquoi ?

Code :

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

1
2
3
4
5
6
7
8
            if (ds != null) {
                Connection conn = ds.getConnection();
                if (conn != null) {
                    connected = true;
                    conn.close();
                    connected = false;
                }
            }

et quand tu fais un conn.getConn() tu en ouvre une nouvelle.

Code :

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

1
2
3
    public Connection getConn() throws SQLException {
        return ds.getConnection();
    }//fin getConn

**Valerio** · 12/04/2005, 13h00

oui en effet, j'avais pas réaliser ca

(hum hum les erreurs les plus grosses sont sous nos yeux)

J'ai donc modifier la classe Connexion :

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
39
40
41
42
43
44
45
46
 
package be.cityform.administration.sql;
 
import java.sql.Connection;
import java.sql.SQLException;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.sql.DataSource;
 
public class Connexion {
 
    /** La référence au DataSource */
    private DataSource ds = null;
 
    /**
     * Initialise la connexion au DataSource
     */
    public void init() {
        try {
            Context initContext = new InitialContext();
            Context envContext = (Context) initContext.lookup("java:/comp/env");
            ds = (DataSource) envContext.lookup("jdbc/postgres");
 
            if (ds != null) {
                Connection conn = ds.getConnection();
            }
        } catch (Exception e) {
            e.printStackTrace();
        }
    }//fin init
 
    /**
     * Retourne la session de connexion
     * 
     * @return Connection
     * @throws SQLException
     */
    public Connection getConn() throws SQLException {
        return ds.getConnection();
    }//fin getConn
 
    public void close () throws SQLException  {
        ds.getConnection().close();
    }//fin close
 
}//fin Connexion

et une requete SQL donne désormais :

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 static float esth_montantProduits(Date date){
	    float montant = -1;
 
        // Récupère la connexion au DataSource
        Connexion conn = new Connexion();
 
 
        PreparedStatement pstmt=null;
        ResultSet rs=null;
        try {
            if (date==null){
                Calendar cal = Calendar.getInstance();
                date = new Date(cal.getTimeInMillis());
            }
       		pstmt = conn.getConn().prepareStatement("SELECT total_produits FROM esth_livre_caisse WHERE date_debut=?;");
       		pstmt.setDate(1, date);
            rs = pstmt.executeQuery();
 
            if(rs.next()){
                montant = rs.getFloat("total_produits");
            }
            rs.close();
            pstmt.close();
        }
        catch(SQLException sqle){
            //TODO voir si on voit le message
            System.err.println(sqle.getMessage());
        }
        finally{
            try{
                conn.close();
            }
            catch(SQLException sqle){}
            conn=null;
        }
        return montant;
	}//fin esth_montantProduitsEnCours

c'est deja un peu plus propre ?

**Alwin** · 12/04/2005, 13h34

Heu non, là tu ouvres 3 connexions.

Une dans la méthode init, une dans la méthode getConn et une troisième dans la méthode close

**Valerio** · 12/04/2005, 13h34

en modifiant un peu le fichier de contexte
en mettant :

Code :

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

1
2
3
4
5
 
    <parameter>
      <name>maxWait</name>
      <value>10000</value>
    </parameter>

j'obtiens, lorsque je rafraichit 2 ou 3 fois, l'exception :

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
39
40
41
 
org.apache.commons.dbcp.SQLNestedException: Cannot get a connection, pool exhausted
	at org.apache.commons.dbcp.PoolingDataSource.getConnection(PoolingDataSource.java:103)
	at org.apache.commons.dbcp.BasicDataSource.getConnection(BasicDataSource.java:540)
	at be.cityform.administration.sql.Connexion.<init>(Connexion.java:24)
	at be.cityform.administration.sql.RequetesSQL.datesLivreCaisse(RequetesSQL.java:618)
	at org.apache.jsp.LivresCaisse.liste_jsp._jspService(liste_jsp.java:89)
	at org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:94)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	at org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:324)
	at org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:292)
	at org.apache.jasper.servlet.JspServlet.service(JspServlet.java:236)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:237)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:157)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:214)
	at org.apache.catalina.core.StandardValveContext.invokeNext(StandardValveContext.java:104)
	at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:520)
	at org.apache.catalina.core.StandardContextValve.invokeInternal(StandardContextValve.java:198)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:152)
	at org.apache.catalina.core.StandardValveContext.invokeNext(StandardValveContext.java:104)
	at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:520)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:137)
	at org.apache.catalina.core.StandardValveContext.invokeNext(StandardValveContext.java:104)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:118)
	at org.apache.catalina.core.StandardValveContext.invokeNext(StandardValveContext.java:102)
	at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:520)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
	at org.apache.catalina.core.StandardValveContext.invokeNext(StandardValveContext.java:104)
	at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:520)
	at org.apache.catalina.core.ContainerBase.invoke(ContainerBase.java:929)
	at org.apache.coyote.tomcat5.CoyoteAdapter.service(CoyoteAdapter.java:160)
	at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:799)
	at org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.processConnection(Http11Protocol.java:705)
	at org.apache.tomcat.util.net.TcpWorkerThread.runIt(PoolTcpEndpoint.java:577)
	at org.apache.tomcat.util.threads.ThreadPool$ControlRunnable.run(ThreadPool.java:683)
	at java.lang.Thread.run(Thread.java:595)
Caused by: java.util.NoSuchElementException: Timeout waiting for idle object
	at org.apache.commons.pool.impl.GenericObjectPool.borrowObject(GenericObjectPool.java:756)
	at org.apache.commons.dbcp.PoolingDataSource.getConnection(PoolingDataSource.java:95)
	... 34 more