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] [Created] (ACCUMULO-2415) Refactor Tables.getNamespaceId to not throw IllegalArgumentException
Date Thu, 27 Feb 2014 00:29:19 GMT
Christopher Tubbs created ACCUMULO-2415:
-------------------------------------------

             Summary: 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
(v6.1.5#6160)

Mime
View raw message