Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-3896

Make AssignmentManager standalone testable by having its constructor take Interfaces rather than a CatalogTracker and a ServerManager

    XMLWordPrintableJSON

Details

    • Task
    • Status: Closed
    • Major
    • Resolution: Won't Fix
    • None
    • None
    • None
    • None
    • noob

    Description

      If we could stand up an instance of AssignmentManager, a core fat class that has a bunch of critical logic managing state transitions, then it'd be easier writing unit tests around its logic. Currently its hard because it takes a ServerManager and a CatalogTracker, but a little bit of work could turn these into Interfaces. SM looks easy to do. Changing CT into an Interface instead might ripple a little through the code base but it'd probably be well worth it.

      Attachments

        Activity

          People

            Unassigned Unassigned
            stack Michael Stack
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: