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

TabletBalancer has an implicit requirement for a specific constructor

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Cannot Reproduce
    • None
    • None
    • master
    • None

    Description

      TabletBalancers are an extension point designed for non-core-Accumulo devs to implement balancers for a specific purpose.

      As noticed in ACCUMULO-3356, there is an unwritten requirement that any concrete implementations of TabletBalancer have a constructor that takes a String object as an argument. This wouldn't be so bad but it's a silent failure that falls back to the DefaultBalancer (that's why we haven't noticed ACCUMULO-3356 for ages).

      TabletBalancer already has an init method which could be used. There are lots of other ways it could be addressed. At a minimum, this should be prominently documented.

      Attachments

        Issue Links

          Activity

            No work has yet been logged on this issue.

            People

              Unassigned Unassigned
              elserj Josh Elser
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: