On Fri, 18 Oct 2024 10:04:55 GMT, Claes Redestad <redes...@openjdk.org> wrote:

> One is to store the `checkedHash` rather than the full `String`. This gets 
> `openCloseZipFile` down to ~910000 ns/op. (`checkedHash` very hot in 
> profiles). There's still a chance for redundant lookups on hash collisions, 
> but this should be rare.

Seems like a resonable trade-off.  Could you take a look at the latest 771488e 
and see if that represents your suggestion here?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/21489#discussion_r1806388657

Reply via email to