Aleksey Shipilev wrote:
Right, Tim. It was just the proof-of-concept "Does IdentityHashMap
tuning worth it?".
I think it is unhelpful to mix actual, concrete suggestions for
performance improvements, with proof-of-concept ideas that break the
specification or security.
Please can people mark clearly in their mail whether they are suggesting
the idea only as "a thought experiment", otherwise somebody may apply
the change without properly understanding the consequences.
I realize the onus is on the committer to ensure it is safe but hiding
such problems is, as I said, unhelpful.
Thanks,
Tim
On Mon, Apr 21, 2008 at 4:50 PM, Tim Ellison <[EMAIL PROTECTED]> wrote:
Sergey Salishev wrote:
It also should be noticed that replacing the IdentityHashMap with HashMap
in
Thread impl gives 4.5x boost on ThreadLocalBench.
But it would be wrong since it opens a security vulnerability.
See:
http://markmail.org/message/rwq3kif5n33wp65m
http://svn.apache.org/viewvc?view=rev&revision=477355
Regards,
Tim
