[jira] [Updated] (HBASE-5328) Small changes to Master to make it more testable

["stack (Updated) (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HBASE-5328?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

stack updated HBASE-5328:
-------------------------

    Attachment: 5328.txt

Need to finish more tests to prove these changes enough (also need to make it so you don't need to know that much about Master's workings writing tests).

M src/main/java/org/apache/hadoop/hbase/catalog/CatalogTracker.java
  (waitForRoot) Remove toxic method  
M src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
  Javadoc and added check of master still running in a few strategic
  places so we don't do infinite loop.
M src/main/java/org/apache/hadoop/hbase/master/HMaster.java
  Refactor moving the instant schema change out to its own method.
  Added flag to indicate when master can receive RPCs.
  (createCatalogTracker, createServerManager, getRemoteInetAddress): put these creations
  out in methods so can override in tests to insert mocks, etc.
A src/test/java/org/apache/hadoop/hbase/master/MockRegionServer.java
  Added a mock regionserver in the master package for testing master
A src/test/java/org/apache/hadoop/hbase/master/Mocking.java
  Utility shared by tests in the master package
A src/test/java/org/apache/hadoop/hbase/master/TestMasterNoCluster.java
  A few tests that put up master and poke it to make it move
  through states.

Patch is not done.  Need to remove the gratuitous changes.  Also need to add some tests that simulate some of the states that Ram and crew are finding.
                
> Small changes to Master to make it more testable
> ------------------------------------------------
>
>                 Key: HBASE-5328
>                 URL: https://issues.apache.org/jira/browse/HBASE-5328
>             Project: HBase
>          Issue Type: Task
>            Reporter: stack
>         Attachments: 5328.txt
>
>
> Here are some small changes in Master that make it more testable.  Included tests stand up a Master and then fake it into thinking that three regionservers are registering making master assign root and meta, etc.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira