Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/HBaseClusterTestCase.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/HBaseClusterTestCase.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/HBaseClusterTestCase.java (working copy) @@ -28,11 +28,17 @@ final boolean miniHdfs; int regionServers; - protected HBaseClusterTestCase() { + /** + * constructor + */ + public HBaseClusterTestCase() { this(true); } - protected HBaseClusterTestCase(int regionServers) { + /** + * @param regionServers + */ + public HBaseClusterTestCase(int regionServers) { this(true); this.regionServers = regionServers; } @@ -37,11 +43,17 @@ this.regionServers = regionServers; } - protected HBaseClusterTestCase(String name) { + /** + * @param name + */ + public HBaseClusterTestCase(String name) { this(name, true); } - protected HBaseClusterTestCase(final boolean miniHdfs) { + /** + * @param miniHdfs + */ + public HBaseClusterTestCase(final boolean miniHdfs) { super(); this.miniHdfs = miniHdfs; this.regionServers = 1; @@ -47,7 +59,11 @@ this.regionServers = 1; } - protected HBaseClusterTestCase(String name, final boolean miniHdfs) { + /** + * @param name + * @param miniHdfs + */ + public HBaseClusterTestCase(String name, final boolean miniHdfs) { super(name); this.miniHdfs = miniHdfs; this.regionServers = 1; Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/HBaseTestCase.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/HBaseTestCase.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/HBaseTestCase.java (working copy) @@ -47,8 +47,9 @@ } protected volatile Configuration conf; - - protected HBaseTestCase() { + + /** constructor */ + public HBaseTestCase() { super(); conf = new HBaseConfiguration(); } @@ -53,7 +54,10 @@ conf = new HBaseConfiguration(); } - protected HBaseTestCase(String name) { + /** + * @param name + */ + public HBaseTestCase(String name) { super(name); conf = new HBaseConfiguration(); } @@ -58,8 +62,9 @@ conf = new HBaseConfiguration(); } + /** {@inheritDoc} */ @Override - protected void setUp() throws Exception { + public void setUp() throws Exception { super.setUp(); this.testDir = getUnitTestdir(getName()); this.localFs = FileSystem.getLocal(this.conf); @@ -68,8 +73,9 @@ } } + /** {@inheritDoc} */ @Override - protected void tearDown() throws Exception { + public void tearDown() throws Exception { if (this.localFs != null && this.testDir != null && this.localFs.exists(testDir)) { this.localFs.delete(testDir); Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestCleanRegionServerExit.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestCleanRegionServerExit.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestCleanRegionServerExit.java (working copy) @@ -43,14 +43,6 @@ } /** - * {@inheritDoc} - */ - @Override - public void setUp() throws Exception { - super.setUp(); - } - - /** * The test * @throws IOException */ Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestCompaction.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestCompaction.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestCompaction.java (working copy) @@ -30,18 +30,6 @@ public class TestCompaction extends HBaseTestCase { static final Log LOG = LogFactory.getLog(TestCompaction.class.getName()); - /** {@inheritDoc} */ - @Override - public void setUp() throws Exception { - super.setUp(); - } - - /** {@inheritDoc} */ - @Override - public void tearDown() throws Exception { - super.tearDown(); - } - /** * Run compaction and flushing memcache * @throws Exception Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHBaseCluster.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHBaseCluster.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHBaseCluster.java (working copy) @@ -57,12 +57,6 @@ cleanup(); } - /** {@inheritDoc} */ - @Override - public void tearDown() throws Exception { - super.tearDown(); - } - private static final int FIRST_ROW = 1; private static final int NUM_VALS = 1000; private static final Text CONTENTS = new Text("contents:"); Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHLog.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHLog.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHLog.java (working copy) @@ -31,12 +31,6 @@ /** JUnit test case for HLog */ public class TestHLog extends HBaseTestCase implements HConstants { - /** {@inheritDoc} */ - @Override - public void setUp() throws Exception { - super.setUp(); - } - /** * @throws IOException */ @@ -101,9 +95,4 @@ } } - /** {@inheritDoc} */ - @Override - public void tearDown() throws Exception { - super.tearDown(); - } } \ No newline at end of file Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHMemcache.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHMemcache.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHMemcache.java (working copy) @@ -55,12 +55,6 @@ this.conf = new HBaseConfiguration(); } - /** {@inheritDoc} */ - @Override - public void tearDown() throws Exception { - super.tearDown(); - } - private Text getRowName(final int index) { return new Text("row" + Integer.toString(index)); } Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHRegion.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHRegion.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHRegion.java (working copy) @@ -43,11 +43,6 @@ static final Logger LOG = Logger.getLogger(TestHRegion.class.getName()); - /** Constructor */ - public TestHRegion() { - super(); - } - /** * Since all the "tests" depend on the results of the previous test, they are * not Junit tests that can stand alone. Consequently we have a single Junit Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHStoreFile.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHStoreFile.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestHStoreFile.java (working copy) @@ -46,8 +46,9 @@ private Configuration conf; private Path dir = null; + /** {@inheritDoc} */ @Override - protected void setUp() throws Exception { + public void setUp() throws Exception { super.setUp(); this.conf = new HBaseConfiguration(); this.fs = FileSystem.getLocal(this.conf); @@ -54,8 +55,9 @@ this.dir = new Path(DIR, getName()); } + /** {@inheritDoc} */ @Override - protected void tearDown() throws Exception { + public void tearDown() throws Exception { if (this.fs.exists(this.dir)) { this.fs.delete(this.dir); } Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestListTables.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestListTables.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestListTables.java (working copy) @@ -35,11 +35,6 @@ new HTableDescriptor("table3") }; - /** constructor */ - public TestListTables() { - super(); - } - /** {@inheritDoc} */ @Override public void setUp() throws Exception { Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestRegionServerAbort.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestRegionServerAbort.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestRegionServerAbort.java (working copy) @@ -43,14 +43,6 @@ } /** - * {@inheritDoc} - */ - @Override - public void setUp() throws Exception { - super.setUp(); - } - - /** * The test * @throws IOException */ Index: src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestTable.java =================================================================== --- src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestTable.java (revision 573634) +++ src/contrib/hbase/src/test/org/apache/hadoop/hbase/TestTable.java (working copy) @@ -25,11 +25,6 @@ /** Tests table creation restrictions*/ public class TestTable extends HBaseClusterTestCase { - /** constructor */ - public TestTable() { - super(true); - } - /** * the test * @throws IOException