<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<html>
<body link="#355491" alink="#4262a1" vlink="#355491" style="background: #e2e2e2; margin: 0; padding: 20px;">
<div>
        <table cellpadding="0" bgcolor="#FFFFFF" border="0" cellspacing="0" style="border: 1px solid #dadada; margin-bottom: 30px; width: 100%; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
                <tbody>
                        <tr>
                                <td>
                                        <table border="0" cellpadding="0" cellspacing="0" bgcolor="#FFFFFF" style="border: solid 2px #ccc; background: #dadada; width: 100%; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
                                                <tbody>
                                                        <tr>
                                                                <td bgcolor="#000000" valign="middle" height="58px" style="border-bottom: 1px solid #ccc; padding: 20px; -moz-border-radius-topleft: 3px; -moz-border-radius-topright: 3px; -webkit-border-top-right-radius: 5px; -webkit-border-top-left-radius: 5px;">
                                                                        <h1 style="color: #333333; font: bold 22px Arial, Helvetica, sans-serif; margin: 0; display: block !important;">
                                                                        <!-- To have a header image/logo replace the name below with your img tag -->
                                                                        <!-- Email clients will render the images when the message is read so any image -->
                                                                        <!-- must be made available on a public server, so that all recipients can load the image. -->
                                                                        <a href="http://community.jboss.org/index.jspa" style="text-decoration: none; color: #E1E1E1">Community</a></h1>
                                                                </td>
                                                        </tr>
                                                        <tr>
                                                                <td bgcolor="#FFFFFF" style="font: normal 12px Arial, Helvetica, sans-serif; color:#333333; padding: 20px; -moz-border-radius-bottomleft: 4px; -moz-border-radius-bottomright: 4px; -webkit-border-bottom-right-radius: 5px; -webkit-border-bottom-left-radius: 5px;"><h3 style="margin: 10px 0 5px; font-size: 17px; font-weight: normal;">
Classloading and caching changes
</h3>
<span style="margin-bottom: 10px;">
reply from <a href="http://community.jboss.org/people/alesj">Ales Justin</a> in <i>JBoss Microcontainer Development</i> - <a href="http://community.jboss.org/message/543110#543110">View the full discussion</a>
</span>
<hr style="margin: 20px 0; border: none; background-color: #dadada; height: 1px;">
<div class="jive-rendered-content"><blockquote class="jive-quote">"the classloader used is in the parent domain"<p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>But this isn't always the case?<br/>Only for this piece:</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><pre class="jive-pre"><code class="jive-code jive-java">      <span style="color: darkgreen;">// Try the before attempt (e.g. from the parent)</span>
      Loader loader = <span style="color: navy;"><strong>null</strong></span>;
      <span style="color: navy;"><strong>if</strong></span> (findInParent)
         loader = findBeforeLoader(name);
      <span style="color: navy;"><strong>if</strong></span> (loader != <span style="color: navy;"><strong>null</strong></span>)
         <span style="color: navy;"><strong>return</strong></span> loader;
</code></pre>
<p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p>
<p>But this could be cached -- if the policy permits it?</p>
<p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p>
<pre class="jive-pre"><code class="jive-code jive-java">      <span style="color: darkgreen;">// Next use any requesting classloader, this will look at everything not just what it exports</span>
      <span style="color: navy;"><strong>if</strong></span> (classLoader != <span style="color: navy;"><strong>null</strong></span>)
      <span style="color: navy;">{</span>
         <span style="color: navy;"><strong>if</strong></span> (trace)
            log.trace(<span style="color: navy;"><strong>this</strong></span> + <span style="color: red;">" trying to load "</span> + name + <span style="color: red;">" from requesting "</span> + classLoader);
         <span style="color: navy;"><strong>if</strong></span> (classLoader.getResourceLocally(name) != <span style="color: navy;"><strong>null</strong></span>)
            <span style="color: navy;"><strong>return</strong></span> classLoader.getLoader();
      <span style="color: navy;">}</span>
</code></pre>
</blockquote><p>That's why it can't update the Cache in the place you suggested.<br/>For a classloader inside the domain it can globally cache, otherwise it can't.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>It should cache in the domain where the classloader lives.<br/>What needs changing is to check that cache according to the parent delegation<br/>rules outside the synchronization block.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>if (matchesParentBefore)<br/>  checkParentCache();<br/>checkOurCache();</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>The difficulty comes in that we would like to add<br/>if (matchesParentAfter)<br/>  checkParentCache();</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>But we can't because the next logical step is to see if we can load from our domain.<br/>This however would work:</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>if (isBlackListedInOurDomain && matchesParentAfter)<br/>  checkParentCache();</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>because we know we won't load it from our domain.</p></div>
<div style="background-color: #f4f4f4; padding: 10px; margin-top: 20px;">
<p style="margin: 0;">Reply to this message by <a href="http://community.jboss.org/message/543110#543110">going to Community</a></p>
        <p style="margin: 0;">Start a new discussion in JBoss Microcontainer Development at <a href="http://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2115">Community</a></p>
</div></td>
</tr>
</tbody>
</table>
</td>
</tr>
</tbody>
</table>
</div>
</body>
</html>