DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2005-05-05 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2004-10-27 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903

WebappClassLoader fails to load class form local repository





--- Additional Comments From [EMAIL PROTECTED]  2004-10-27 14:26 ---
This is a reproducible real problem.

Our server uses tomcat 5.0 as a servlet engine. When it runs multiple worker
threads to execute a JavaScritpt (using Rhino) on the server side, it will
occasionally but consistently throw NoClassDefFoundError on a class in Rhino
library, which is located in the WEB-INF\lib directory. 

We eventually figure out if two threads start to load same class concurrently
for WEB-INF/lib, one of them will fail if 

a) the first thread manages to call defineClass on the resource entry and set
the binaryContent to null; 

b) when the second adds its entry to the HashMap, it will find an entry for that
 class already exists, so it will return the existing entry (the one whose
binaryContent is null); 

For the second part, it has been reported by some other people. Please see the
following link

http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html

It would be better to replace the code with single checked locking and make it
thread-safe.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2004-10-27 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903

WebappClassLoader fails to load class form local repository

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED



--- Additional Comments From [EMAIL PROTECTED]  2004-10-27 14:30 ---
You're not supposed to fork threads in your webapp.

As for the double check thing, try to post productive stuff, thanks.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2004-10-27 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903

WebappClassLoader fails to load class form local repository





--- Additional Comments From [EMAIL PROTECTED]  2004-10-27 14:33 ---
It's well and good that you're not supposed to fork threads, but let's get
real -- this is something that code needs to do more often than the strict
letter of J2EE lets on.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2004-10-27 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903

WebappClassLoader fails to load class form local repository





--- Additional Comments From [EMAIL PROTECTED]  2004-10-27 14:54 ---

You're not supposed to fork threads in your webapp.

Oh, this is the simpler fix, thanks.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2004-10-27 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903

WebappClassLoader fails to load class form local repository

[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|5.0.28  |5.5.3



--- Additional Comments From [EMAIL PROTECTED]  2004-10-27 15:14 ---
The J2EE spec forbids it, and experience shows it can cause many tricky problems.

The change for entry.loadedClass == null is good, since I added the
entry.binaryContent = null without seeing that the check would need an update.
This was introduced in 5.0.3.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2004-10-27 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903

WebappClassLoader fails to load class form local repository





--- Additional Comments From [EMAIL PROTECTED]  2004-10-27 16:20 ---
Thanks Remy. It is forbidden to create threads in ejb container, and it is not
recommended to spawn threads in servlet container, but don't recall this is
strictly forbidden by the servlet spec (I might be wrong here).

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 31903] - WebappClassLoader fails to load class form local repository

2004-10-26 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=31903.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31903

WebappClassLoader fails to load class form local repository

[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|WebappClassLoader fails to  |WebappClassLoader fails to
   |load class form local   |load class form local
   |repository  |repository



--- Additional Comments From [EMAIL PROTECTED]  2004-10-26 23:37 ---
Are you reporting an actual problem, or is it just reading of the source code ?

The first part could make sense: I added entry.binaryContent = null after the
rest, so the condition might be wrong.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]