accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
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
             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

View raw message