Pavel, why no-need-test? It even has related issue for test development. I will update the label for now
|
|
|
URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/9ca10ab155e3
User: lana
Date: 2012-12-28 18:30:44 +0000
|
|
|
URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/9ca10ab155e3
User: coffeys
Date: 2012-12-20 17:35:22 +0000
|
|
|
SQE/OK for accepting this fix
|
|
|
**may be necessary for SPECjbb2012 numbers.
|
|
|
EVALUATION
signed binaries pushed to closed repo :
http://closedjdk.us.oracle.com/jdk7u/jdk7u-dev/jdk/make/closed/rev/af86159206d1
|
|
|
EVALUATION
backport, signed binaries to follow..
|
|
|
EVALUATION
The suggested patch replaces the usage of Hashtable w/ ConcurrentHashMap.
This will introduce an incompatiblity in serialization format as CryptoPermissions class are declared to be serializable and the field "perms" (which used to be Hashtable) are part of the serialized bytes.
The easiest approach would be to bump out the serialVersionUID to indicate the incompatibility between different JDK releases after making this Hashtable -> ConcurrentHashMap performance enhancement.
Otherwise, we may have to employ some custom serialization methods to maintain the old serialization format to ensure compatibility.
|
|
|