accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Miller (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-4681) Create WeakReference Map to replace Table.ID constructor
Date Tue, 08 Aug 2017 20:18:00 GMT


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

> Create WeakReference Map to replace Table.ID constructor
> --------------------------------------------------------
>                 Key: ACCUMULO-4681
>                 URL:
>             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

View raw message