"[EMAIL PROTECTED]" wrote : | You mention "severe performance problems". Do you have any more info about what you were doing and what you saw? I initially classified the JIRA issue as "Minor" as that's kind of standard for a performance improvement patch, but any more info you have would help to better prioritize the issue. |
I have to perform lots of searches on Sets which causes substantial use of iteration. I profiled my code and found out that most of the time was spent on size() method that calls getChildren() (size() is called twice in hasNext() and again once in next() ). The new version acquires node children only once when constructing a new Iterator and then utilizes them in iteration (no calls to getChildren() in hasNext() and next() ). This is very similar to your current implementation of CachedMapImpl. The change improved the performance of my program from about 70 seconds to 6 seconds. As my program is also performing other things than just iterating through Sets, I guess that the performance of CachedSetImpl must have improved at least by a factor of 20. BTW, have you considered implementing a new POJO Cache such that it would not use TreeCache as its backend? It seems that most of the time in TreeCacheAop is spent on updating the tree structure when some reference in cached POJO is updated... If you are considering such thing, I would gladly share my experiences during the specification. -Jussi- View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3919047#3919047 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=3919047 ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642 _______________________________________________ JBoss-user mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/jboss-user
