You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by "Michael Miller (JIRA)" <ji...@apache.org> on 2017/07/19 14:53:00 UTC

[jira] [Updated] (ACCUMULO-4681) Create WeakReference Map to replace Table.ID constructor

     [ https://issues.apache.org/jira/browse/ACCUMULO-4681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Michael Miller updated ACCUMULO-4681:
-------------------------------------
    Description: 
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.

> 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
>         Environment: 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.
>            Reporter: Michael Miller
>            Priority: Minor
>
> 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)