nobeh commented on PR #534: URL: https://github.com/apache/tomcat/pull/534#issuecomment-1203780743
> already implements the relevant logic, so we don't need to do anything extra with this property Yes, that's correct. However, Tomcat uses a "MethodHandle" invocation to call that method. When a system is under load, through profiling, we've observed that invoking the method handle to evaluate to `false` could actually be an expensive operation. This is one of the main motivations for this change. > but may also be force Yes. Thanks for the catch. I will update this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
