On 03/06/2013 03:42 PM, Mandy Chung wrote:
LoggerResourceBundleRace.java: I think what you really want is to
add a new test that sets a context class loader to a class loader
that finds the resource bundle for a logger that a system class
loader can't find. I suggest to leave this test as it is and then
add a new test to exercise the context class loader search of a
resource bundle as a separate RFE that will improve the test coverage.
Leaving the existing test as is not an option unless we change the
test to run in othervm as I had on my first webrev. The bundles are
not found otherwise. Hence the change to set the context classloader.
I'm confused. Is the resource bundle
"LoggerResourceBundleRace$MyResources" in the classpath as this test
class is located? Should the system class loader be able to find
them? This test shouldn't depend on the stack search. Shouldn't the
thread context classloader be null and then use the system class
loader? Is another test set the thread context classloader in the
jtreg run? Does the test pass if you run it directly (not via jtreg)?
I figured it out. I was running jtreg locally with -s -- samevm. In
this case, the bundles can't be found via the system classloader.
However, both directly running the tests and running in jprt which uses
jtreg -a (agentvm) works fine. I can leave the test as is.
Thanks,
Jim
--
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.g...@oracle.com