accumulo-notifications mailing list archives

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


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

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:
>             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

View raw message