Using non-parallel custom class loaders for Layer configurations

David M. Lloyd david.lloyd at redhat.com
Fri Sep 9 21:26:11 UTC 2016


On 09/09/2016 02:05 PM, Mandy Chung wrote:
>
>> On Sep 9, 2016, at 8:44 AM, David M. Lloyd <david.lloyd at redhat.com> wrote:
>>
>> On 09/08/2016 06:48 PM, Mandy Chung wrote:
>>>
>>>> On Sep 8, 2016, at 3:29 PM, David M. Lloyd <david.lloyd at redhat.com> wrote:
>>>>
>>>> Would it be possible to include a method like this (pretty old patch I had laying around):
>>>>
>>>> diff --git a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
>>>> index 1bb1580..3def10e 100644
>>>> --- a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
>>>> +++ b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
>>>> @@ -469,6 +477,15 @@ public abstract class ClassLoader {
>>>>        return lock;
>>>>    }
>>>>
>>>> +    /**
>>>> +     * Determine whether this class loader is parallel-capable.
>>>> +     *
>>>> +     * @return {@code true} if the class loader is parallel-capable, {@code false} otherwise
>>>> +     */
>>>> +    protected final boolean isParallelCapable() {
>>>> +        return ParallelLoaders.isRegistered(getClass());
>>>> +    }
>>>> +
>>>
>>> This seems a fine method to add.  I think it can be a public method (I don’t see any reason it has to be protected)
>>
>> Great!  Since it's such a small method though, would it be possible to just roll this change into some other changeset?  I can't really contribute the change in any form other than a patch right now.
>
> I file a RFE and Brent Christian has agreed to work on it and will go through JDK 9 FC approval request.
>   https://bugs.openjdk.java.net/browse/JDK-8165793

Thanks!
-- 
- DML


More information about the jigsaw-dev mailing list