Review Board 1.7.22


Fix for HBASE-6364

Review Request #6521 - Created Aug. 10, 2012 and submitted

nkeywal
Reviewers
hbase
hbase-git
 TestHBaseClient to be in the final commit
Test_HBASE_6364 / HBaseRecoveryTestingUtility not for the final commit
unit tests ok
Total:
13
Open:
6
Resolved:
7
Dropped:
0
Status:
From:
Description From Last Updated Status
Please add a short javadoc for this class. Ted Yu Aug. 10, 2012, 3:24 p.m. Open
Please use EnvironmentEdgeManager instead of System. Ted Yu Aug. 10, 2012, 3:24 p.m. Open
Should we check that nbRS is greater than 2 ? Ted Yu Aug. 10, 2012, 3:24 p.m. Open
with 'j < 2', only 2 slots in RS array are filled. If this is intentional, please add a comment ... Ted Yu Aug. 10, 2012, 3:24 p.m. Open
Please add a comment to say what is considered a dirty region server. Ted Yu Aug. 10, 2012, 3:24 p.m. Open
Needs class comment to say why/when it'd be used Michael Stack Aug. 18, 2012, 10:03 p.m. Open
Review request changed
Updated (Dec. 24, 2012, 2:58 p.m.)
  • changed from pending to submitted