Re: Review request 7190657 Modify getDriver() to call Thread.currentThread().getContextClassLoader();

2012-08-10 Thread David Schlosnagle
Lance, There are a couple of other uses of DriverManager.getCallerClassLoader() that do no fall back to Thread.currentThread().getContextClassLoader() if the caller class loader is null (bootstrap loader). Should these be updated as well? >From >http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/tip/

hg: jdk8/tl/jdk: 4 new changesets

2012-08-10 Thread lana . steuck
Changeset: b3b0d75cb117 Author:katleman Date: 2012-08-09 18:02 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b3b0d75cb117 Added tag jdk8-b51 for changeset e865efbc7105 ! .hgtags Changeset: da8649489aff Author:lana Date: 2012-08-10 10:15 -0700 URL: http://hg.

hg: jdk8/tl/hotspot: 15 new changesets

2012-08-10 Thread lana . steuck
Changeset: 86a687be3f02 Author:amurillo Date: 2012-07-27 16:51 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/hotspot/rev/86a687be3f02 7187463: new hotspot build - hs24-b19 Reviewed-by: jcoomes ! make/hotspot_version Changeset: 594dff5e3c2e Author:johnc Date: 2012-07-17 11

hg: jdk8/tl/jaxp: Added tag jdk8-b51 for changeset dc1ea77ed9d9

2012-08-10 Thread lana . steuck
Changeset: bd3c00d57614 Author:katleman Date: 2012-08-09 18:01 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/bd3c00d57614 Added tag jdk8-b51 for changeset dc1ea77ed9d9 ! .hgtags

hg: jdk8/tl/langtools: 2 new changesets

2012-08-10 Thread lana . steuck
Changeset: 23032c78b2d1 Author:katleman Date: 2012-08-09 18:02 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/23032c78b2d1 Added tag jdk8-b51 for changeset c4cd4cab2220 ! .hgtags Changeset: 1d2db0e5eabc Author:lana Date: 2012-08-10 10:14 -0700 URL: http

hg: jdk8/tl/jaxws: Added tag jdk8-b51 for changeset 1a70b6333ebe

2012-08-10 Thread lana . steuck
Changeset: f62bc618122e Author:katleman Date: 2012-08-09 18:01 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/f62bc618122e Added tag jdk8-b51 for changeset 1a70b6333ebe ! .hgtags

hg: jdk8/tl: Added tag jdk8-b51 for changeset 57c0aee73090

2012-08-10 Thread lana . steuck
Changeset: 8d24def5ceb3 Author:katleman Date: 2012-08-09 18:00 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/rev/8d24def5ceb3 Added tag jdk8-b51 for changeset 57c0aee73090 ! .hgtags

hg: jdk8/tl/corba: Added tag jdk8-b51 for changeset 9b0f841ca9f7

2012-08-10 Thread lana . steuck
Changeset: 80689ff9cb49 Author:katleman Date: 2012-08-09 18:00 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/80689ff9cb49 Added tag jdk8-b51 for changeset 9b0f841ca9f7 ! .hgtags

hg: jdk8/tl/jdk: 3 new changesets

2012-08-10 Thread valerie . peng
Changeset: 114fbbeb8f75 Author:valeriep Date: 2012-08-10 13:08 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/114fbbeb8f75 7107613: scalability bloker in javax.crypto.CryptoPermissions Summary: Changed the type of field "perms" from Hashtable to ConcurrentHashMap. Reviewed-by

Re: PING: [PATCH FOR REVIEW] System Zlib Support

2012-08-10 Thread Alan Bateman
On 10/08/2012 18:40, Andrew Hughes wrote: - Original Message - Hi Andrew Alan mentioned the other day that the build (with your latest patch, with zlib-dev) failed at pack200 when he tried to turn it on on linux. Can your double check on your system? What kind of failure? All webrevs

Re: PING: [PATCH FOR REVIEW] System Zlib Support

2012-08-10 Thread Andrew Hughes
- Original Message - > Hi Andrew > > Alan mentioned the other day that the build (with your latest patch, > with zlib-dev) failed at pack200 > when he tried to turn it on on linux. Can your double check on your > system? > What kind of failure? All webrevs I've posted pass a complete bu

Review request 7190657 Modify getDriver() to call Thread.currentThread().getContextClassLoader();

2012-08-10 Thread Lance Andersen - Oracle
Looking for a reviewer for the following change: - add a call to Thread.currentThread().getContextClassLoader() to DriverManager.getDriver() - Remove the synchronized block for the same call in getConnection() Thank you. Best Lance localhost:sql lanceandersen$ hg diff DriverManager.java diff

Re: PING: [PATCH FOR REVIEW] System Zlib Support

2012-08-10 Thread Xueming Shen
Hi Andrew Alan mentioned the other day that the build (with your latest patch, with zlib-dev) failed at pack200 when he tried to turn it on on linux. Can your double check on your system? -Sherman On 08/09/2012 06:31 AM, Andrew Hughes wrote: Just checking on the status of this. Am I ok to

hg: jdk8/tl/jdk: 2 new changesets

2012-08-10 Thread sean . mullan
Changeset: 57b5025548d6 Author:mullan Date: 2012-08-10 09:12 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/57b5025548d6 7187962: sun.security.pkcs11.P11DSAKeyFactory.implTranslatePublicKey doesn't check if params is null Reviewed-by: valeriep ! src/share/classes/sun/securi