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

Christopher Tubbs resolved ACCUMULO-2415.
-----------------------------------------
    Resolution: Not A Problem

In spite of the javadoc, it actually throws TableNotFoundException instead.

> Refactor Tables.getNamespaceId to not throw IllegalArgumentException
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-2415
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2415
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Christopher Tubbs
>            Priority: Minor
>
> 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.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to