[GitHub] tomcat issue #72: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=59901

2017-08-09 Thread zemian
Github user zemian commented on the issue: https://github.com/apache/tomcat/pull/72 Okay Mark. At this point, I am not sure I have a full grasp of the solution you wanted yet. I will study little more and revisit this issue later. --- If your project is set up for it, you can reply

[GitHub] tomcat issue #72: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=59901

2017-08-09 Thread markt-asf
Github user markt-asf commented on the issue: https://github.com/apache/tomcat/pull/72 My point is that with the o.a.c.webresources.Cache in place, this solution offers little additional benefit. I'm not suggesting creating dependencies from Jasper to Catalina. On closer

[GitHub] tomcat issue #72: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=59901

2017-08-07 Thread zemian
Github user zemian commented on the issue: https://github.com/apache/tomcat/pull/72 Hi Mark, > Tomcat provides (by default) a static content cache in the resources layer so I do wonder if there is much benefit in caching the content in the JSP engine as well. Are

[GitHub] tomcat issue #72: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=59901

2017-08-07 Thread markt-asf
Github user markt-asf commented on the issue: https://github.com/apache/tomcat/pull/72 Tomcat provides (by default) a static content cache in the resources layer so I do wonder if there is much benefit in caching the content in the JSP engine as well. The proposed cache appears