accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-2087) Tables.getNamespaces NPE's on missing table ID
Date Tue, 24 Dec 2013 04:10:51 GMT

    [ 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)

Mime
View raw message