[JAVA] JSP geheugenlek bij vullen MYSQL db

[Zandor]

ik heb het volgende probleempje:
op mijn servertje (llinux debian woody) ben ik af en toe lekker aan het worstelen. voor de duidelijkheid: ben geen profesional ook redelijke linux noob nog.

ik merk echter aan het geheugen dat als ik 10000 maal dezelfde string in een mysql database zet, er vele mb's (50 mb extra) in gebruik genomen worden, terwijl het toch dezelfde string is en een eenvoudige herhaalactie.

hoe zorg ik dat het geheugen ook weert vrijgegeven wordt: ik zie via meminfo dat het geheugengebruik ook niet meer omlaag gaat. als ik bijvoorbeeld de loop 200000 maal laat herhalen loop ie echt vast en krijg dan een java outofmemory error.

voorbeeldcode waarbij ik het probleem heb.
Kan iemand mij helpen en zeggen wat ik fout doe?

code:

<%@ 
page language="java"
import="java.sql.*" 
import="java.lang.Object"
%>

<%
Class.forName("com.mysql.jdbc.Driver").newInstance();
Connection db = DriverManager.getConnection("jdbc:mysql://localhost:3306/twb","-----","----");
%>

<%int van = 0;
int tot = 10000;
String content=new String();
String opslaan=new String();
String invul = new String();%>

<%
for (int loep = van; loep < tot ; loep++) { %>
<%=(loep)%>&nbsp
                <% content = "bladibladiblabladibladiblabladibladiblabladibl"; %>

<%
invul = content;
opslaan="INSERT INTO twb(content) VALUES (?)";
PreparedStatement ps = db.prepareStatement(opslaan);
ps.setString(1, invul);
int rs = ps.executeUpdate();
ps.close();
}

db.close();
System.gc();

%>
</body></html>

[Onno]

Beetje merkwaardig dat je elke keer opnieuw een prepared statement maakt. Het idee van die dingen is nou juist dat je ze kunt hergebruiken.

Dat zou verder geen lek moeten veroorzaken trouwens.

[Zandor]

Onno schreef op donderdag 03 februari 2005 @ 21:54:
Beetje merkwaardig dat je elke keer opnieuw een prepared statement maakt.

check

Dat zou verder geen lek moeten veroorzaken trouwens.

check

maaruh het toevoegen van System.gc(); lijkt bij mij ook geen verschil uit te maken.

[Johnny]

Heb je al een memory profiler gebruikt? Daarmee kun je precies zien welke class al dat geheugen op vreet.

[ACM]

Wellicht helpt het nog om na elke executie een commit te doen? Maar op zich is het raar dat ie daar op lekt, het zou idd niet moeten.

Je kan waarschijnlijk het beste de boel testen door zo min mogelijk in de for-loop te plaatsen. Imho alleen de ps.setString en ps.executeUpdate-functies en dus evt een cs.commit(). De rest zou er allemaal buiten moeten blijven, sowieso is dat het beste voor de performance

[Zandor]

Johnny schreef op donderdag 03 februari 2005 @ 22:20:
Heb je al een memory profiler gebruikt? Daarmee kun je precies zien welke class al dat geheugen op vreet.

nee, ik zal gaan uitzoeken hoe dat werkt enezo, ken het niet.

[Zandor]

ACM schreef op donderdag 03 februari 2005 @ 22:24:
Wellicht helpt het nog om na elke executie een commit te doen? Maar op zich is het raar dat ie daar op lekt, het zou idd niet moeten.

ga ik proberen.

Je kan waarschijnlijk het beste de boel testen door zo min mogelijk in de for-loop te plaatsen. Imho alleen de ps.setString en ps.executeUpdate-functies en dus evt een cs.commit(). De rest zou er allemaal buiten moeten blijven, sowieso is dat het beste voor de performance

ga ik ook proberen.


kan ik trouwens zelf nog iets anderes forceren mbt die garbage collector?

[ACM]

Ik zou niet te veel proberen met de garbage collector te werken. In principe zou die helemaal niet nodig moeten zijn namelijk.

[Verwijderd]

Nog een opmerking, je hoeft ook geen new string() voor een string declaratie op te roepen. Bij de toekenning van string literal (= ook 'gewoon' een String object) wordt dat String object dan nodeloos weer weggegooid.

ps

Je kunt ook beter engelse namen in je code gebruiken