[infinispan-issues] [JBoss JIRA] Commented: (ISPN-1065) Use a better mechanism to parse config files
Pete Muir (JIRA)
jira-events at lists.jboss.org
Tue Jun 14 05:42:23 EDT 2011
[ https://issues.jboss.org/browse/ISPN-1065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12608382#comment-12608382 ]
Pete Muir commented on ISPN-1065:
---------------------------------
I think we should strongly consider writing a Stax parser by hand. Previous experience tells me that this will result in a much more elegant architecture for our code (much simpler to keep up to date as a result) and will perform much better. I would also propose using a tool to read the docs from the schema and create a config doc, meaning all that needs keeping up to date is the schema and parser.
> Use a better mechanism to parse config files
> --------------------------------------------
>
> Key: ISPN-1065
> URL: https://issues.jboss.org/browse/ISPN-1065
> Project: Infinispan
> Issue Type: Enhancement
> Components: Configuration
> Affects Versions: 4.2.1.FINAL
> Reporter: Manik Surtani
> Assignee: Vladimir Blagojevic
> Labels: jaxb
> Fix For: 5.1.0.BETA1, 5.1.0.Final
>
>
> The current codebase heavily relies on JAXB to:
> 1. Generate a XSD schema from a Configuration bean
> 2. Generate a parser to use with an XML configuration
> This is known to be slow, and can significantly add to start up time:
> {quote}
> We did some analysis of our AS6 descriptor code which was using JAXB (800ms), compared to straight stax (10ms)
> {quote}
> A possible solution could be to still use JAXB to generate the schema, but to also write a Maven plugin which would generate a STAX parser based on the XSD schema and Configuration bean at build time. Infinispan would then use this STAX parser rather than a JAXB generated parser, which would be a lot faster.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the infinispan-issues
mailing list