jkesselm commented on PR #113: URL: https://github.com/apache/xalan-java/pull/113#issuecomment-1839622955
The biggest reason we need the cache, ***if *** I remember correctly, is semantics rather than performance. XPath/XSLTC make some assertions about node identity that need to be true across multiple references to the document, and that becomes hard to achieve if the document() call retrieves a new set of node objects each time. But that behavior only needs to be true across one stylesheet's execution. We don't want to drop things from the cache while in use, but after all references to them have gone away they can be uncached and GC'd (at the cost of possibly reloading them later.) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
