[icedtea-web] RFC: Patch to fix PR895
omajid at redhat.com
Mon Mar 12 11:31:58 PDT 2012
On 03/12/2012 02:20 PM, Deepak Bhole wrote:
> * Deepak Bhole <dbhole at redhat.com> [2012-03-12 14:13]:
>> * Omair Majid <omajid at redhat.com> [2012-03-09 20:08]:
>>> On 03/09/2012 05:19 PM, Deepak Bhole wrote:
>>>> Attached patch fixes PR895:
>>>> "IcedTea-Web searches for missing classes on each loadClass or findClass"
>>> The bug doesn't seem to have any additional information. Is there a test
>>> case or a reproducer? Do you know why the server is being hammered? If
>>> the actual issue is under our control (for example, we are attempting to
>>> re-download stuff we already downloaded), it might be better to fix this
>>> underlying issue.
>> Yes, this one:
Please do add this to
http://icedtea.classpath.org/wiki/IcedTea-Web-Tests as well.
>> Without the patch it takes 2+ minutes to run.. with patch, it is down to
>> about 15 seconds.
Do you know what this test is doing? While the patch looks fine, I would
hate for us to paper over the problem rather than fixing it.
>> While writing the above, I realized that it is really just CBCL that we
>> want to fix. What about moving the logic just into CBCL? See attached
> annndddd... here is the patch
Just one request: can you please add a unit test for this classloader so
we can verify that this behaviour does not regress later?
More information about the distro-pkg-dev