You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commons-dev@ws.apache.org by "Takahide Nogayama (JIRA)" <ji...@apache.org> on 2008/02/01 15:31:12 UTC

[jira] Created: (WSCOMMONS-300) Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.

Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.
---------------------------------------------------------------------------------------------

                 Key: WSCOMMONS-300
                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-300
             Project: WS-Commons
          Issue Type: Improvement
          Components: AXIOM
         Environment: Windows
            Reporter: Takahide Nogayama


When I read XML document which has too many text elements, I found performance down. I think the reader.getProperty("IsDataHandlerAware") call for each text element is one of the overhead for this case.

My question is "Should we really ask the property for each text element?"

If answer is No, then  StAXBuilder can cache the property to avoid many getProperty calls. I will post a patch file to solve this performance issue.



-- 
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: commons-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: commons-dev-help@ws.apache.org


[jira] Assigned: (WSCOMMONS-300) Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.

Posted by "Rich Scheuerle (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WSCOMMONS-300?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rich Scheuerle reassigned WSCOMMONS-300:
----------------------------------------

    Assignee: Rich Scheuerle

> Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.
> ---------------------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-300
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-300
>             Project: WS-Commons
>          Issue Type: Improvement
>          Components: AXIOM
>         Environment: Windows
>            Reporter: Takahide Nogayama
>            Assignee: Rich Scheuerle
>         Attachments: patch.txt
>
>
> When I read XML document which has too many text elements, I found performance down. I think the reader.getProperty("IsDataHandlerAware") call for each text element is one of the overhead for this case.
> My question is "Should we really ask the property for each text element?"
> If answer is No, then  StAXBuilder can cache the property to avoid many getProperty calls. I will post a patch file to solve this performance issue.

-- 
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: commons-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: commons-dev-help@ws.apache.org


[jira] Resolved: (WSCOMMONS-300) Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.

Posted by "Rich Scheuerle (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WSCOMMONS-300?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rich Scheuerle resolved WSCOMMONS-300.
--------------------------------------

    Resolution: Fixed

> Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.
> ---------------------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-300
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-300
>             Project: WS-Commons
>          Issue Type: Improvement
>          Components: AXIOM
>         Environment: Windows
>            Reporter: Takahide Nogayama
>            Assignee: Rich Scheuerle
>         Attachments: patch.txt
>
>
> When I read XML document which has too many text elements, I found performance down. I think the reader.getProperty("IsDataHandlerAware") call for each text element is one of the overhead for this case.
> My question is "Should we really ask the property for each text element?"
> If answer is No, then  StAXBuilder can cache the property to avoid many getProperty calls. I will post a patch file to solve this performance issue.

-- 
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: commons-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: commons-dev-help@ws.apache.org


[jira] Updated: (WSCOMMONS-300) Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.

Posted by "Takahide Nogayama (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WSCOMMONS-300?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Takahide Nogayama updated WSCOMMONS-300:
----------------------------------------

    Attachment: patch.txt

patch to StAXBuilder.java

> Many reader.getProperty() calls for each text element in StAXBuilder causes performance down.
> ---------------------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-300
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-300
>             Project: WS-Commons
>          Issue Type: Improvement
>          Components: AXIOM
>         Environment: Windows
>            Reporter: Takahide Nogayama
>         Attachments: patch.txt
>
>
> When I read XML document which has too many text elements, I found performance down. I think the reader.getProperty("IsDataHandlerAware") call for each text element is one of the overhead for this case.
> My question is "Should we really ask the property for each text element?"
> If answer is No, then  StAXBuilder can cache the property to avoid many getProperty calls. I will post a patch file to solve this performance issue.

-- 
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: commons-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: commons-dev-help@ws.apache.org