[ 
https://issues.apache.org/jira/browse/ACCUMULO-2087?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13856112#comment-13856112
 ] 

Christopher Tubbs commented on ACCUMULO-2087:
---------------------------------------------

Tables is not supposed to be public API. This is just a helper/utility class to 
read from ZooKeeper. I haven't worked through all of it yet to see the pitfalls 
of concurrency... but my guess is that Tables is prone to these sorts of issues 
everywhere. It probably shouldn't throw IllegalArgumentException... it should 
probably pass the null through instead of making a new String if null, and be 
handled by the caller.

> Tables.getNamespaces NPE's on missing table ID
> ----------------------------------------------
>
>                 Key: ACCUMULO-2087
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2087
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Josh Elser
>            Priority: Critical
>             Fix For: 1.6.0
>
>
> If the user provides a tableID which doesn't exist to getNamespace, the 
> implementation will NPE if a table with that ID doesn't exist.
> It should throw a proper exception and be noted in the javadoc as such.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to