Dhamoder Nalla
Dhamoder Nalla
> @dhanalla This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply...
> This PR seems OK. It would have been easier to review if the whitespace wasn't all different. Consider adding [jerboaa@054efe5](https://github.com/jerboaa/jdk8u-dev/commit/054efe518681e6334b700c8a236fa4c508db640d) to make this more aligned with JDK 11u. Finally,...
> > This PR seems OK. It would have been easier to review if the whitespace wasn't all different. Consider adding [jerboaa@054efe5](https://github.com/jerboaa/jdk8u-dev/commit/054efe518681e6334b700c8a236fa4c508db640d) to make this more aligned with JDK 11u....
> I'm surprised you think this is suitable for 8u. It's not low risk, and it's more of a performance fix than a bug, and it's P4. JDK 8u is...
> If we are going to backport this, then I think we should use the following approach in order to keep the risk manageable (and to keep in sync with...
> This seems OK now modulo some whitespace changes. The best course of action would be to compare the `KeepAliveCache.java` file to the version from `jdk11u` and fix the whitespace...
Thanks @jerboaa & @phohensee for reviewing this PR. I added the JDK8u fix request tag to the bug as the next step. Please let me know if there is anything...
> @gnu-andrew 8293562: The approval request has been approved. Thanks @gnu-andrew for reviewing this PR.
Thanks @rzhao271 for creating this PR. As part of our security compliance initiative, we are also working to address undocumented APIs, as these are utilized within the OpenJDK repository.