JCgH4164838Gh792C124B5 commented on pull request #489: URL: https://github.com/apache/struts/pull/489#issuecomment-855547053
Hello Apache Struts Team. This PR ended up involving many more files than originally planned, due to the number of unit test changes that were made. This is a first draft attempt to find a solution to the behaviour issue seen with tag pooling application servers, but others who review it may be able to point to a better alternative. Interactive tests with Glassfish 5.0.1 appear to show the idea works, and the Showcase application seems to also work properly too. The call overhead of these changes may exceed the benefits, but should make the JSP tag behaviour correct (or mostly correct) when tag class instances are re-used by tag pools. If anyone has suggestions on improvements, or alternatives, please let me know. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
