Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 0.19.2
    • Fix Version/s: 0.90.0
    • Component/s: None
    • Labels:
      None

      Description

      This is refactor to Server Manager class from HBASE-1017

      I separate it for reasons:

      • Its better to have several small patchs and apply them iterativly then one great path
      • I fu..** tired from synchronising w/ SVN (this class changes frequently), you can saw 10 patches in HBASE-1017

      We need this refactoing for reasons:

      • Server Manager looks like shi**.. bad thing...
      • is every time harder to make any chnages
      • it is becoming more ugly every time........

      What changes are done:

      ServerManager has mapping:

      • serverName 2 serverInfo,
      • serverAddr 2 serverInfo,
      • serverName 2 load,
      • load 2 severName

      1) serverName 2 load - not necessary if you have serverName 2 serverInfo
      2) All mappings are encapsulated in ServersInfo class (inner class of ServerManager)
      3) ServersInfo has operations for adding, updating and removing information of HRS

      + some code in RegionServer is puted in synchronised block... cause it is working with synchronised map...

      Note: this task is to make code much much more clear.. and it's not going to change logic, so no much problem is going appear

        Attachments

        1. HBASE-1422_v3.patch
          35 kB
          Evgeny Ryabitskiy
        2. HBASE-1422_v2.patch
          26 kB
          Evgeny Ryabitskiy
        3. HBASE-1422.patch
          26 kB
          Evgeny Ryabitskiy

          Issue Links

            Activity

              People

              • Assignee:
                apparition Evgeny Ryabitskiy
                Reporter:
                apparition Evgeny Ryabitskiy
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: