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

Michael Miller commented on ACCUMULO-4681:
------------------------------------------

Ok I will create a static "of" method for Namespace.  Do you think it needs an 
internal WeakHashMap as well?  I guess it woudln't hurt in the case where if 
someone was ever to create a lot of namespaces.

> Create WeakReference Map to replace Table.ID constructor
> --------------------------------------------------------
>
>                 Key: ACCUMULO-4681
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4681
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Michael Miller
>            Assignee: Michael Miller
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> Taken from feedback on the PR #279:
> Could maybe avoid duplicates by making constructor (of Table.ID) private and 
> doing Table.ID.of(tableId), which draws from an internal WeakReference map.
> If the object deduplication in KeyExtent is still valid, this can be pushed 
> down to the Table.ID and Namespace.ID classes, replacing the optimization in 
> KeyExtent.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to