[ 
https://issues.apache.org/jira/browse/XERCESJ-1296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12843962#action_12843962
 ] 

Ludger Bünger commented on XERCESJ-1296:
----------------------------------------

Hi Michael,

To verify this, I profiled the same situation again and it looks much better 
now.

However I discovered another hotspot in getBaseURI which has been shadowed by 
this issue before.

Shall I open a new issue or attach a performance fix for getBaseURI to this one?

> ElementNSImpl.getBaseURI has exponential execution time relative to the depth 
> of the node in the document
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: XERCESJ-1296
>                 URL: https://issues.apache.org/jira/browse/XERCESJ-1296
>             Project: Xerces2-J
>          Issue Type: Improvement
>          Components: DOM (Level 3 Core)
>    Affects Versions: 2.9.1
>            Reporter: Ludger Bünger
>            Assignee: Michael Glavassevich
>            Priority: Minor
>             Fix For: 2.10.0
>
>         Attachments: ElementNSImplPatch.txt
>
>
> When using the Namespace-aware implementation, getBaseURI upon ElementNSImpl 
> unnecessarily calls getBaseURI twice upon its parent node (once upon "parent" 
> and once upon "ownerNode") resulting in an exponential execution time 
> relative to the depth of the node (since the number of calls to their 
> respective parent method doubles with each level of invocation).
> Elena Litani solved this already for ElementImpl, the same patch should be 
> applied to ElementNSImpl too.
> Solution: Apply the same patch to ElementNSImpl Elena Litani already applied 
> to ElementImpl in svn build 319554 (see 
> http://svn.apache.org/viewvc/xerces/java/trunk/src/org/apache/xerces/dom/ElementImpl.java?r1=319502&r2=319554)

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to