[ 
https://issues.apache.org/jira/browse/WSCOMMONS-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12919617#action_12919617
 ] 

Kasun Gajasinghe commented on WSCOMMONS-556:
--------------------------------------------

Ok, That change is good.

Regarding Revision #1006309,
The method getParentOwnerDocument(), was introduced to get the 
most-grand-parent node of the given node, and set the UserData object to that 
node, rather than setting it to the current Node object. This is what Xerces 
dom implementation does. So, we worked on it and changed NodeImpl#setUserData 
and NodeImpl#getUserData accordingly. We used the above method to get the 
OwnerNode. But there were some issues, and we found another almost same 
solution to it. i.e. basically stopping creation of a Hashtable instance for 
every Node creation. Now it creates the userData hashtable instance only 
setUserData is invoked. Performance improvement by this change is bigger, as 
these setUserData and getUserData will not be invoked frequently. Apparently, 
this method was left behind!

> Performance improvement for axiom-dom module
> --------------------------------------------
>
>                 Key: WSCOMMONS-556
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-556
>             Project: WS-Commons
>          Issue Type: Improvement
>          Components: AXIOM
>    Affects Versions: Axiom 1.2.9
>         Environment: Java 1.5
>            Reporter: Kasun Gajasinghe
>            Assignee: Andreas Veithen
>             Fix For: Axiom 1.2.10
>
>         Attachments: axiom-dom-556-patch2.diff, axiom-perf-improvement.diff, 
> FINAL-times_for_large_samples_with_changes.png, 
> FINAL-times_for_small_samples_with_changes.png
>
>
> This article published [0] on April 2010, shows that Axis2 has some major 
> performance issues compared to Metro and CXF.  An analysis suggested that 
> most of the problems were with the axiom-dom module's methods, along with 
> Rampart. So, first we went on improving the axiom-dom module. Here's a patch 
> for the improvements we did.
> Final assessments suggest considerable level improvement.  Graphs for a 
> sample run of requests with small and large samples will be added to the 
> files section.
> [0] http://www.ibm.com/developerworks/java/library/j-jws14/index.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to