[ 
https://issues.apache.org/jira/browse/ATLAS-1026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15379697#comment-15379697
 ] 

David Kantor commented on ATLAS-1026:
-------------------------------------

[~shwethags] The instance of StoreBackedTypeCache.Context used in get() not a 
static instance, it is a local variable.  The class definition is declared as 
private static so that it is static nested class with no reference to an 
enclosing instance.  It is declared private as it's usage is limited to the 
internal implementation of StoreBackedTypeCache.get() and its child methods.

> StoreBackedTypeCache issues
> ---------------------------
>
>                 Key: ATLAS-1026
>                 URL: https://issues.apache.org/jira/browse/ATLAS-1026
>             Project: Atlas
>          Issue Type: Bug
>            Reporter: Shwetha G S
>            Assignee: David Kantor
>
> After ATLAS-1010, StoreBackedTypeCache introduces cyclic calls - 
> StoreBackedTypeCache.getTypeFromStore() calls TypeSystem.defineTypes() which 
> inturn calls TypeSystem.has() which in turn calls 
> StoreBackedTypeCache.getTypeFromStore().
> Also, StoreBackedTypeCache.Context is static which will not work
> DefaultTypeCache.put() and putAll() should use putIfAbsent()



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to