[ACCUMULO-4681] Create WeakReference Map to replace Table.ID constructor

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Assignee: Michael Miller
• Reporter: Michael Miller
• Created: 18/Jul/17 21:55
• Updated: 17/Aug/17 14:36
• Resolved: 17/Aug/17 14:34

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.

Issue Links

• links to: GitHub Pull Request #292