RFR (XS): 8214208: Nestmate package validation logging/exception should include classloader information

David Holmes david.holmes at oracle.com
Tue Nov 27 21:28:57 UTC 2018


Thanks Harold and Coleen.

David

On 28/11/2018 5:56 am, coleen.phillimore at oracle.com wrote:
> +1
> Coleen
> 
> On 11/27/18 2:18 PM, Harold David Seigel wrote:
>> Hi David,
>>
>> These changes look good.
>>
>> Thanks, Harold
>>
>>
>> On 11/22/2018 7:14 AM, David Holmes wrote:
>>> On 22/11/2018 8:58 pm, David Holmes wrote:
>>>> Missed another test that needs updating ... will revise tomorrow.
>>>
>>> Revised tonight - webrev updated in placed with additional changes for
>>>
>>> /test/jdk/java/lang/reflect/Nestmates/TestReflectionAPI.java
>>>
>>> Thanks,
>>> David
>>>
>>>> David
>>>>
>>>> On 22/11/2018 7:55 pm, David Holmes wrote:
>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8214208
>>>>> webrev: http://cr.openjdk.java.net/~dholmes/8214208/webrev/
>>>>>
>>>>> This is a simple change to add classloader information to the 
>>>>> logging and exceptions related to obtaining the nest-host within 
>>>>> the VM.
>>>>>
>>>>> This makes it simpler to see when tests fail due to nested types 
>>>>> being loaded by different classloaders.
>>>>>
>>>>> Of course the test had to be updated to account for the additional 
>>>>> information.
>>>>>
>>>>> Testing: runtime/Nestmates
>>>>>           tier 1 - 2
>>>>>
>>>>> Thanks,
>>>>> David
>>
> 


More information about the hotspot-runtime-dev mailing list