commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 32369] - [discovery] Doesn't work with ClassLoaders that do not support getResource()
Date Wed, 01 Dec 2004 21:42:38 GMT
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=32369>.
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=32369





------- Additional Comments From rsitze@apache.org  2004-12-01 22:42 -------
Ah, another voice.. thank you.

I want to note that the patch in question uses loadClass to determine if the 
resource is available, but in fact returns the resource name [along with the 
loader that found it]... the class isn't returned, isn't used, and it isn't 
[today] cached.

On the other hand: for performance, the existing implementation defers 
resolution of classes and other artifacts it needs for as long as possible 
[won't claim perfection, just a goal and general scheme].  So... for the cases 
where you grab the first one available the current scheme should be sufficient 
even with the patch... and I think this is more to your point.

The if(url = null) idea is more in line with where I would be willing to go... 
if a resource isn't available then the load should fail very quickly [assuming 
classloaders are implemented for performance].

I haven't looked at the code in a while, so I'm not on top of the 
ramifications of this... suppose I need to refresh.

Let me start looking at the code.

<ras>


-- 
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: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message