Memory leak with tagfiles - re-use JasperLoader?

2007-12-10 Thread Korn, Christoph
Hello Tomcat (Jasper) developers, in our project we have a serious memory problem (perm gen space) related to JSP tag files: The class generated from the tag file gets loaded for every JSP the tag is placed on. So if you put such a tag on i.e. 8000 JPSs, the tag class will seriously pollute the

Re: Memory leak with tagfiles - re-use JasperLoader?

2007-12-10 Thread Remy Maucherat
On Mon, 2007-12-10 at 11:30 +0100, Korn, Christoph wrote: I perfectly understand that loading each JSPs via a different ClassLoader is necessary if you want to be able to reload the JSP-class, but if the reloading-feature is turned off (development=false), I don't see the need for a new

Re: Memory leak with tagfiles - re-use JasperLoader?

2007-12-10 Thread Antonio Petrelli
2007/12/10, Remy Maucherat [EMAIL PROTECTED]: This is a hack. This may work great for you, and that's the whole point of using OSS, but this is not a solution for others. Here I disagree: at Struts we were discussing one big problem with JSP (that depends on the specification, not the

AW: Memory leak with tagfiles - re-use JasperLoader?

2007-12-10 Thread Korn, Christoph
This is a hack. This may work great for you, and that's the whole point of using OSS, but this is not a solution for others. Sure this is a dirty hack and I haven't tested it for more than five minutes. As I have no deep understanding of the Tomcat code and what side effects the patch might

DO NOT REPLY [Bug 44047] New: - Provide a way for Tomcat to serve up error pages when too busy

2007-12-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://issues.apache.org/bugzilla/show_bug.cgi?id=44047. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.

error with ant download for 6.0.14

2007-12-10 Thread Joe Bohn
I'm trying to build 6.0.14 and I'm hitting the following problems with the ant download step. It appears that the version of several dependencies referenced in build.properties.default are no longer available:

DO NOT REPLY [Bug 43839] - I think the implement is difference with the document in function CoyoteAdapter.postParseRequest

2007-12-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://issues.apache.org/bugzilla/show_bug.cgi?id=43839. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.

DO NOT REPLY [Bug 43839] - I think the implement is difference with the document in function CoyoteAdapter.postParseRequest

2007-12-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://issues.apache.org/bugzilla/show_bug.cgi?id=43839. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.

Re: error with ant download for 6.0.14

2007-12-10 Thread Filip Hanik - Dev Lists
this is the one we are using http://www.eclipse.org/downloads/download.php?file=/eclipse/downloads/drops/R-3.3.1-200709211145/eclipse-JDT-3.3.1.zip Filip Joe Bohn wrote: I'm trying to build 6.0.14 and I'm hitting the following problems with the ant download step. It appears that the version

implementing shared tomcat hosting

2007-12-10 Thread Leo Wraith
[EMAIL PROTECTED] Hi, I attempting to provide JSP/Servlet (shared JVM), support for multiple sites through my hosting company. My configuration is as follows: OS: CentOS Tomcat: 5.5.9 JDK: 1.5+ Right now I have it so that a website that has a *.jsp will work

svn commit: r603074 - /tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

2007-12-10 Thread markt
Author: markt Date: Mon Dec 10 14:24:40 2007 New Revision: 603074 URL: http://svn.apache.org/viewvc?rev=603074view=rev Log: Fix bug 44041. A small sync is required to prevent attempts to load the same class twice. Modified: tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

svn commit: r603075 - /tomcat/tc6.0.x/trunk/STATUS.txt

2007-12-10 Thread markt
Author: markt Date: Mon Dec 10 14:26:40 2007 New Revision: 603075 URL: http://svn.apache.org/viewvc?rev=603075view=rev Log: Propose fix for 44041 Modified: tomcat/tc6.0.x/trunk/STATUS.txt Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL:

svn commit: r603076 - /tomcat/current/tc5.5.x/STATUS.txt

2007-12-10 Thread markt
Author: markt Date: Mon Dec 10 14:27:22 2007 New Revision: 603076 URL: http://svn.apache.org/viewvc?rev=603076view=rev Log: Propose fix for 44041 Modified: tomcat/current/tc5.5.x/STATUS.txt Modified: tomcat/current/tc5.5.x/STATUS.txt URL:

DO NOT REPLY [Bug 44041] - WebappClassLoader duplicate class definition under high Load if hasExternalRepositories is true

2007-12-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://issues.apache.org/bugzilla/show_bug.cgi?id=44041. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.

Re: svn commit: r603074 - /tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

2007-12-10 Thread Filip Hanik - Dev Lists
if String objects are kept in a constants pool, then one would synchronized (name) { clazz = super.findClass(name); } to allow concurrent loading of different classes, Filip [EMAIL PROTECTED] wrote: Author: markt Date: Mon Dec 10

svn commit: r603110 - /tomcat/tc6.0.x/trunk/STATUS.txt

2007-12-10 Thread fhanik
Author: fhanik Date: Mon Dec 10 17:10:13 2007 New Revision: 603110 URL: http://svn.apache.org/viewvc?rev=603110view=rev Log: enhancement comment Modified: tomcat/tc6.0.x/trunk/STATUS.txt Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL:

Re: svn commit: r603074 - /tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

2007-12-10 Thread Peter Rossbach
+1 for that... Peter Am 11.12.2007 um 01:53 schrieb Filip Hanik - Dev Lists: if String objects are kept in a constants pool, then one would synchronized (name) { clazz = super.findClass(name); } to allow concurrent loading of