[
https://issues.apache.org/jira/browse/WSCOMMONS-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12919607#action_12919607
]
Andreas Veithen commented on WSCOMMONS-556:
-------------------------------------------
It needs to be revised because if one wants to eliminate the unnecessary object
creation overhead completely, then having a combined method that returns an
array is not an option (anyway, returning an array in this case is considered
by many people as an anti-pattern; it should return a QName object instead).
What I'm going to do is to revert that part of the change and fix the
getLocalName and getPrefix methods properly. In that case, the only overhead is
the double lookup of the ':' character (if the caller uses both methods). This
is acceptable.
> 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.
