Skip to content
This repository has been archived by the owner on Aug 8, 2020. It is now read-only.

Add caching features into Transform filter #4

Merged
merged 3 commits into from
Mar 4, 2016
Merged

Add caching features into Transform filter #4

merged 3 commits into from
Mar 4, 2016

Conversation

furfurylic
Copy link
Owner

Refers #3.

Main target:

  • Transform filters cache documents which are resolved through URIResolvers.
  • Multiple Transform instances share one cache pool.
  • Transform filters can be configured not to do caching.

Not so sure:

  • Caching is done not only through URIResolvers, but also through EntityResolvers.
  • The cache holds documents by soft references, not strong references.

Add caching features into Transform filter
c0820b1 
Transform filter stores documents resolved by a URIResolver
which it holds.
But as of now the cache is not shared by other Transform instances.
@furfurylic furfurylic changed the title [WIP] Add caching features into Transform filter Add caching features into Transform filter Mar 4, 2016
@furfurylic
Copy link
Owner Author

Eventually, caching through EntityResolvers is supported. This will improve performance in cases where many original source documents refer one same DTD files.

furfurylic pushed a commit that referenced this pull request Mar 4, 2016
Merge pull request #4 from furfurylic/transform-cache
bfadb81 
Add caching features into Transform filter
@furfurylic furfurylic merged commit bfadb81 into master Mar 4, 2016
@furfurylic furfurylic deleted the transform-cache branch March 4, 2016 12:32
@furfurylic furfurylic mentioned this pull request Mar 4, 2016
Closed
furfurylic added a commit that referenced this pull request Oct 22, 2017
@furfurylic
Merge pull request #4 from furfurylic/transform-cache
77f5da9 
Add caching features into Transform filter
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@furfurylic