[
https://issues.apache.org/jira/browse/SHINDIG-816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12661825#action_12661825
]
Henning Schmiedehausen commented on SHINDIG-816:
------------------------------------------------
I was a bit time-constrained this week, sorry.
I tested with JDK-6 (which has a Stax implementation included) and with JDK-5 +
woodstox and JDK-6 and woodstox. All three combinations pass all unit tests.
Personally I prefer the JDK-6 + woodstox combination because it has the best
performance.
More comments later. sorry.
> Stax based parser for GadgetSpec and MessageBundle
> --------------------------------------------------
>
> Key: SHINDIG-816
> URL: https://issues.apache.org/jira/browse/SHINDIG-816
> Project: Shindig
> Issue Type: Improvement
> Components: Gadget Rendering Server (Java)
> Affects Versions: 1.0.x-incubating
> Reporter: Henning Schmiedehausen
> Attachments: RELEASE-NOTES, svn-stax.patch
>
>
> This is a large patch which I don't expect to go in without some discussion.
> However, we have to start at some point.
> As I wrote before (and those of you following my mails to the dev list
> probably know), I need some features that the current, DOM based parser for
> GadgetSpec and related to this, for MessageBundle, can not deliver. I need to
> reconstruct gadget specs from the objects, passing through other namespaced
> elements and a number of additional requirements which, in the end, led to a
> complete new implementation of the GadgetSpec parser using Stax.
> The attached RELEASE-NOTES file describes in ample detail the problems that I
> have encountered and the quirks I discovered while dissecting the existing
> parser.
> And, while it does not really matter with good caching, preliminary tests
> show that the parser is measurably faster than the old DOM parser. If it were
> possible to stream XML into it, there would be even more performance
> improvements.
> Given the fact, that we will soon see other namespaces in the gadget spec
> (the whole templating stuff), this should be much easier and more readable
> doable using this parser approach.
> A code review ist at http://codereview.appspot.com/11676
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
