[ACCUMULO-2415] Refactor Tables.getNamespaceId to not throw IllegalArgumentException

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Not A Problem
• Assignee: Unassigned
• Reporter: Christopher Tubbs
• Created: 27/Feb/14 00:28
• Updated: 09/Jun/21 21:01
• Resolved: 09/Jun/21 21:01

Description

ACCUMULO-2087 introduced throwing IllegalArgumentException if a table does not exist when looking up it's namespace. This caused bug ACCUMULO-2393, which handles that case.

However, this implementation is very brittle, because it relies on a runtime exception, which could hide other errors. It's also throwing an IllegalArgumentException when really, the argument is perfectly valid.

This code should be refactored to handle these circumstances better, in a way that doesn't hide potential bugs. Because this is internal implementation details and a utility method, and not public API, we have some flexibility here, but we need to ensure we have the correct behavior for every caller that uses this method.

Issue Links

• relates to: ACCUMULO-2087 Tables.getNamespaces NPE's on missing table ID
• relates to: ACCUMULO-2393 DeleteTableDuringSplitIT failed