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

Josh Elser commented on ACCUMULO-2087:
--------------------------------------

bq. Tables is not supposed to be public API

I know that, but it's still a public static method that was used by tests, so I 
consider it to have some sensibility.

bq. it should probably pass the null through instead of making a new String if 
null, and be handled by the caller.

:cringe: I don't really care for that practice, but if that's what the rest of 
the code is doing.

bq. Mind if I re-assign this to myself

Go for it. That's why I unassigned it from myself. If you come up with 
something better, feel free to revert the bandaid.

> 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
>            Assignee: Christopher Tubbs
>            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