Github user osma commented on the pull request:

    https://github.com/apache/jena/pull/119#issuecomment-169611251
  
    Oh right, I didn't notice `getOrFill`.
    
    So am I right that the consensus would be to
    1. switch to one of the Jena Cache implementations instead of using 
java.util.LinkedHashMap (CacheUnbounded seems the best choice to me, as there 
is no reason to limit the cache size, it will be short-lived anyway as it only 
applies to a single query execution)
    2 use `getOrFill` to handle cache misses
    
    I can do those changes.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to