Thanks again Paul. Could you sponsor the change? As far as I can tell, I'd need to add a fix request now, but I don't have access to issue tracker. Thanks, Daniel
wt., 16 mar 2021 o 18:59 Hohensee, Paul <[email protected]> napisał(a): > > Looks good! :) > > -----Original Message----- > From: Daniel Jeliński <[email protected]> > Date: Tuesday, March 16, 2021 at 9:49 AM > To: "Hohensee, Paul" <[email protected]> > Cc: "[email protected]" <[email protected]>, > "[email protected]" <[email protected]> > Subject: RE: [11u] RFR 8259886: Improve SSL session cache performance and > scalability > > Thanks Paul for your review and for the hint. > > Updated webrev: https://djelinski.github.io/8259886-11u/webrev2/index.html > > compared to original, changes to make/test/BuildMicrobenchmark.gmk > were dropped because file does not exist in jdk11 > compared to previous webrev, CacheBench was re-added. > > Testing: Linux x86_64 tier1 and tier2. > Thanks, > Daniel > > pon., 15 mar 2021 o 18:09 Hohensee, Paul <[email protected]> napisał(a): > > > > The changes to Cache.java look fine, but please include CacheBench.java. > > I'd like to see 11u to stand on its own without reference to later > > releases, plus I believe the 11u maintainers prefer to backport as much of > > a patch as possible. > > > > Thanks, > > Paul > > > > -----Original Message----- > > From: security-dev <[email protected]> on behalf of Daniel > > Jeliński <[email protected]> > > Date: Tuesday, March 9, 2021 at 3:37 PM > > To: "[email protected]" <[email protected]> > > Cc: "[email protected]" <[email protected]> > > Subject: [11u] RFR 8259886: Improve SSL session cache performance and > > scalability > > > > Hi, > > Please review this 11u backport; this is the same patch as for head, > > except for microbenchmark makefile changes that did not apply because > > the file doesn't exist in 11u, and the actual microbenchmark, which > > would only add weight for no benefit. > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8259886 > > webrev: https://djelinski.github.io/8259886-11u/webrev/index.html > > > > Testing: Linux x86_64 tier1 and tier2. > > > > Thanks, > > Daniel > > >
