Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-2415

Refactor Tables.getNamespaceId to not throw IllegalArgumentException

    Details

    • Type: Improvement
    • Status: Open
    • Priority: 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.

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated: