Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0
    • Component/s: None
    • Labels:
      None

      Description

      While working on ACCUMULO-3238, I noticed a few places where we use the ambiguous variable:

      String table

      This is problematic since it is not clear what the variable is actually referring to. It can be referring to either a tableId or tableName both of which are Strings but different in values Zookeeper. Typically the actual value of the variable can be derived from surrounding code but this is not always the case.

        Issue Links

          Activity

          Hide
          milleruntime Michael Miller added a comment -

          After ACCUMULO-4699 is fixed, refactoring of String -> Table.ID for sourceTableId in SequentialWorkAssigner will need to be done.

          Show
          milleruntime Michael Miller added a comment - After ACCUMULO-4699 is fixed, refactoring of String -> Table.ID for sourceTableId in SequentialWorkAssigner will need to be done.
          Hide
          ctubbsii Christopher Tubbs added a comment -

          I see some commits against this. Are there more expected, or can this be closed?

          Show
          ctubbsii Christopher Tubbs added a comment - I see some commits against this. Are there more expected, or can this be closed?

            People

            • Assignee:
              milleruntime Michael Miller
              Reporter:
              milleruntime Michael Miller
            • Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 1h 10m
                1h 10m

                  Development