https://issues.apache.org/bugzilla/show_bug.cgi?id=53869

--- Comment #11 from Mark Thomas <ma...@apache.org> ---
The test case used to evaluate this patch is flawed. It fails to take account
of the call to rootJspCtxt.getELContext() and does not compare the performance
of the new implementation for a simple JSP page that does not use EL and does
not have nested tags. When these additional issues are taken into consideration
the proposed patch significantly impacts performance for these simple pages.
Performance (as measured by this test case) is 1.5 to 2 times slower for simple
pages.

However, it is not all doom and gloom. A small tweak to the proposed patch to
use lazy instantiation for the EL context and performance is (as near as makes
no difference) back to where it was for simple pages and still noticeably
improved for the more complex case.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to