Mailing-List: contact hbase-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: hbase-dev@hadoop.apache.org
Message-ID: <986408009.1242266685532.JavaMail.jira@brutus>
Date: Wed, 13 May 2009 19:04:45 -0700 (PDT)
From: "Evgeny Ryabitskiy (JIRA)" <jira@apache.org>
To: hbase-dev@hadoop.apache.org
Subject: [jira] Created: (HBASE-1422) Refactor to Server Manager
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Refactor to Server Manager
--------------------------

                 Key: HBASE-1422
                 URL: https://issues.apache.org/jira/browse/HBASE-1422
             Project: Hadoop HBase
          Issue Type: Sub-task
    Affects Versions: 0.19.2
            Reporter: Evgeny Ryabitskiy
            Assignee: Evgeny Ryabitskiy
            Priority: Blocker
             Fix For: 0.20.0


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

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.