Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce XSL Template Caching #77

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

baszero
Copy link
Contributor

@baszero baszero commented May 20, 2016

This pull request actually does NOT yet MAKE USE of the caching, but it adapts the BasicXMLResource so that it is prepared for it. This pull request is 100% backwards compatible. So if this pull request gets merged into master, there is no difference to before.

The Cache implementation could be done in the new class XslTemplatesCache, this is just an example. Of course also another cache can be used.

An example of how to introduce actual caching:

  • Extend (subclass) the BasicXMLResource and overwrite the getTransformerHandler() method where you either return a cached XSL object or create a new one (as today)

A full example is explained in #67 (comment)

backwards compatible).
- new class XslTemplatesCache with high performance cache
- How to use the cache: extend BasicXMLResource and overwrite the method
getTransformerHandler()
- Example for how to use Cache:
wyona#67 (comment)
@baszero
Copy link
Contributor Author

baszero commented May 20, 2016

Little side note: if this pull request gets merged, you might correct the wrong indentation in the method getTransformedInputStream(). thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant