[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411441#comment-13411441 ] Mark Struberg commented on EXTCDI-298: -- please don't cache classes. This has a huge impact. The easy solution is to not run in a doPrivileged block if no SecurityManager is available. Will fix this. [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411445#comment-13411445 ] Thomas Andraschko commented on EXTCDI-298: -- Is there another solution for the class cache? 60ms is much - also with profiling overhead. [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411454#comment-13411454 ] Mark Struberg commented on EXTCDI-298: -- Can you please test now? I've at least removed the doPrivileged for you. It will now only get executed if a SecurityManager is present. For fixing the loadClassForName I would need a stacktrace. And the number of times this gets called. A cache as you did it is not possible since org.mypackage.MyClass might be different for WebAppA and a WebAppB... [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411460#comment-13411460 ] Thomas Andraschko commented on EXTCDI-298: -- Will test it later, thanks! loadClassForName is called arround 9600 / request. That's because i have 1-5 links / row in my repeater with 2000 items. Maybe a className/outcome cache is possible? [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411465#comment-13411465 ] Thomas Andraschko commented on EXTCDI-298: -- Also i know that the classes could be different in WebAppA and WebAppB. Therefore i added the classloader, too :) [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411549#comment-13411549 ] Mark Struberg commented on EXTCDI-298: -- Therefore i added the classloader, too :) ok, then it's fine. But still it's much better to prevent invoking the whole chain if possible. [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411554#comment-13411554 ] Gerhard Petracek commented on EXTCDI-298: - @ thomas: based on the details you provided recently, we need imo a different cache. i'll have a look at the details. [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411557#comment-13411557 ] Thomas Andraschko commented on EXTCDI-298: -- Would it be possible to cache NavigationCase/outcome in CodiNavigationHandler? I already done a patch but don't know if there are other drawbacks then. [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411563#comment-13411563 ] Gerhard Petracek commented on EXTCDI-298: - yes - that's the place where we need a cache (or at least near to it) - as i said: i've to check the details. [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (EXTCDI-298) [perf] reduce overhead in ClassUtils
[ https://issues.apache.org/jira/browse/EXTCDI-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411637#comment-13411637 ] Thomas Andraschko commented on EXTCDI-298: -- ok. if you think a outcome/navigationCase cache in the NavigationHandler is ok, just ping me and i could provide a patch. [perf] reduce overhead in ClassUtils Key: EXTCDI-298 URL: https://issues.apache.org/jira/browse/EXTCDI-298 Project: MyFaces CODI Issue Type: Improvement Reporter: Thomas Andraschko Assignee: Mark Struberg Attachments: EXTCDI-298.patch As discussed with gerhard, i removed the #doPriviliged which improved the method execution duration from 50ms to 0ms (30.000 invocations / request) I also added a class cache which improves loadClassForName from 62ms to 6ms for 9600 invocations. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
