Github user BrynCooke commented on the issue:

    https://github.com/apache/tinkerpop/pull/567
  
    1. ManagedConcurrentValueMap is not a map. I don't really know why we use 
this. but computeIfAbsent is not present on this class.
    2. Guava LoadingCache and MoreExecutors.directExecutor() is the correct way 
of solving this problem. It allows soft values and does all the syncronisation 
stuff for us. Is there a reason why guava is not used by Tinkerpop?
    
    If we want avoid guava we need to have a striped lock on script, and to 
create our own direct executor.



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to