Accumulo
  1. Accumulo
  2. ACCUMULO-2415

Refactor Tables.getNamespaceId to not throw IllegalArgumentException

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      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.

        Issue Links

          Activity

          There are no comments yet on this issue.

            People

            • Assignee:
              Unassigned
              Reporter:
              Christopher Tubbs
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:

                Development