RFR: 8002070 Remove the stack search for a resource bundle for Logger to use
Mandy Chung
mandy.chung at oracle.com
Wed Mar 13 15:40:55 UTC 2013
Great, thanks! I'll push it for you.
Mandy
On 3/13/2013 8:28 AM, Jim Gish wrote:
> Mandy,
>
> The CCC request has been approved. Please push the changeset at
> http://cr.openjdk.java.net/~jgish/Bug8002070-RemoveResourceBundleStackSearch.patch/
> <http://cr.openjdk.java.net/%7Ejgish/Bug8002070-RemoveResourceBundleStackSearch.patch/>
>
>
> Thanks,
> Jim
>
> On 03/08/2013 05:51 PM, Mandy Chung wrote:
>> On 3/8/2013 2:39 PM, Jim Gish wrote:
>>> I've update the webrev with the suggested changes:
>>> http://cr.openjdk.java.net/~jgish/Bug8002070-RemoveResourceBundleStackSearch/
>>> <http://cr.openjdk.java.net/%7Ejgish/Bug8002070-RemoveResourceBundleStackSearch/>
>>>
>>
>> Thumbs up. Thanks for adding the comment in the test. I'd suggest
>> to take out the statement about "stack walk search" since this may be
>> confusing to others who read it as that code will no longer exist.
>> Please add this bug ID to @bug.
>>
>> thanks
>> Mandy
>>
>>> Thanks,
>>> Jim
>>>
>>>
>>> On 03/08/2013 03:41 PM, Mandy Chung wrote:
>>>> On 3/5/2013 11:12 AM, Jim Gish wrote:
>>>>>> For the LoggerResourceBundleRace test then does it have to run in
>>>>>> its own VM?
>>>>> Because we no longer search up the stack for the bundles, this
>>>>> test fails unless run in its own vm. However, to avoid this, I'll
>>>>> try to change the test to set the context classloader.
>>>>
>>>> I discussed with Jim. In jtreg samevm mode, the testclasses are
>>>> not in the classpath and thus the system classloader can't find the
>>>> resource bundle. Before the fix, the test finds the resource
>>>> bundle using the classloader on the stack which isn't the intent.
>>>>
>>>> Mandy
>>>
>>> --
>>> Jim Gish | Consulting Member of Technical Staff | +1.781.442.0304
>>> Oracle Java Platform Group | Core Libraries Team
>>> 35 Network Drive
>>> Burlington, MA 01803
>>> jim.gish at oracle.com
>
> --
> Jim Gish | Consulting Member of Technical Staff | +1.781.442.0304
> Oracle Java Platform Group | Core Libraries Team
> 35 Network Drive
> Burlington, MA 01803
> jim.gish at oracle.com
More information about the core-libs-dev
mailing list