[jira] [Commented] (CAMEL-3999) stax and sax to parse a xml document

["Claus Ibsen (Commented) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CAMEL-3999?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13143897#comment-13143897 ] 

Claus Ibsen commented on CAMEL-3999:
------------------------------------

I would suggest use the package name org.apache.camel.component.stax and have all the logic in there. This is how the other components is.
Also at Apache we discourage having @author tags in the source, so they should be removed in the patch.
Also consider if you have sufficient unit tests. Also for inputs that are not a file etc. And if there is an empty XML document, etc.
                
> stax and sax to parse a xml document
> ------------------------------------
>
>                 Key: CAMEL-3999
>                 URL: https://issues.apache.org/jira/browse/CAMEL-3999
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-spring
>            Reporter: Romain Manni-Bucau
>            Priority: Minor
>         Attachments: CAMEL-3999-camel-stax.patch
>
>
> to parse a xml document without loading it can be useful.
> i had a try here http://code.google.com/p/rmannibucau/source/browse/#hg%2Fcamel%2Fcamel-stax%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fcamel%2Fstax%2Fcomponent to be able to use a sax content handler to parse a document using the stax xml stream reader. In this component the sax content handler is used as an api but the functionnality is the streaming using stax.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira