[ACCUMULO-2594] Improve TabletLocator.getInstance() to check tableId points to a table which exists

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Invalid
• Affects Version/s: 1.5.0
• Component/s: client
• Assignee: Unassigned
• Reporter: Ara Ebrahimi
• Created: 31/Mar/14 18:53
• Updated: 07/Apr/15 03:21
• Resolved: 06/Apr/15 21:35

Description

If you pass an invalid tableId (or in my case tableName instead of tableId) TabletLocator won't complain.

TabletLocator tabletLocator = TabletLocator.getInstance(instance, new Text(tableId));

Next if you use the returned tabletLocator to locate a tablet it just returns an empty list, never complaining that the tableId was invalid:

TabletLocator.TabletLocation tabletLocation = tabletLocator.locateTablet(rowKey.getText(), false, false, credentials.toThrift());

Ideally TabletLocator.getInstance() should guard against invalid/non-existing tableIds.