RFR 8029891 : Deadlock detected in java/lang/ClassLoader/deadlock/GetResource.java

[Peter Levart]

On 05/12/2015 01:43 AM, Brent Christian wrote:
> Hi, Mandy.  Thanks for having a look.
>
> On 5/11/15 12:09 PM, Mandy Chung wrote:
>> ~/WORK/8029891/plevartI'd like Daniel's suggestion to have the load*
>> method to putAll entries in one go after the input reader/stream/xml
>> file is loaded and parsed rather than adding one entry at a time.
>
> I also like the idea.
>
>> Taking another look at this deadlock issue and the compatibility
>> concerns, I wonder if we should keep this change as a special
>> implementation for system properties rather than having this change to
>> java.util.Properties class.  Properties is a Hashtable which specifies
>> the fast-fail behavior (throwing ConcurrentModificationException for
>> concurrent update).  There are other issues specific to system
>> properties we want to clean up (e.g. read-only system property, private
>> system property to JDK but not visible to public etc).
>>
>> Any thought?
>
> I like this idea, too. :)
>
> One thought:
> In the current fix, clone() and serialization make use of 
> package-private methods.  This could present some difficulties if 
> system properties would use its own Properties subclass that would 
> live outside java.util.
>
> -Brent

Do you have an example where you would like to access/override one of 
those methods? They are designed to be a private contract between 
Properties and Hashtable.

Regards, Peter