[
https://issues.apache.org/jira/browse/ATLAS-1026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15379697#comment-15379697
]
David Kantor edited comment on ATLAS-1026 at 7/15/16 5:13 PM:
--------------------------------------------------------------
[~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.
So I don't understand your comment on about this not working, please advise.
was (Author: dkantor):
[~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)
