You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by la...@apache.org on 2013/01/27 06:40:48 UTC
svn commit: r1439026 - /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerMetrics.java

Author: larsh
Date: Sun Jan 27 05:40:48 2013
New Revision: 1439026

URL: http://svn.apache.org/viewvc?rev=1439026&view=rev
Log:
HBASE-7681 Addendum, close tables in TestRegionServerMetrics

Modified:
    hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerMetrics.java

Modified: hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerMetrics.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerMetrics.java?rev=1439026&r1=1439025&r2=1439026&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerMetrics.java (original)
+++ hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerMetrics.java Sun Jan 27 05:40:48 2013
@@ -149,6 +149,8 @@ public class TestRegionServerMetrics {
     metricsHelper.assertCounterGt("totalRequestCount", requests + 39, serverSource);
     metricsHelper.assertCounterGt("readRequestCount", readRequests + 9, serverSource);
     metricsHelper.assertCounterGt("writeRequestCount", writeRequests + 29, serverSource);
+
+    table.close();
   }
 
   @Test
@@ -176,6 +178,8 @@ public class TestRegionServerMetrics {
     metricsHelper.assertGauge("putsWithoutWALCount", 1, serverSource);
     long minLength = row.length + cf.length + qualifier.length + val.length;
     metricsHelper.assertGaugeGt("putsWithoutWALSize", minLength, serverSource);
+
+    t.close();
   }
 
   @Test
@@ -203,6 +207,8 @@ public class TestRegionServerMetrics {
     metricsRegionServer.getRegionServerWrapper().forceRecompute();
     metricsHelper.assertGauge("storeCount", stores +1, serverSource);
     metricsHelper.assertGauge("storeFileCount", storeFiles + 1, serverSource);
+
+    t.close();
   }
 
   @Test
@@ -237,6 +243,8 @@ public class TestRegionServerMetrics {
     metricsRegionServer.getRegionServerWrapper().forceRecompute();
     metricsHelper.assertCounter("checkMutateFailedCount", 1, serverSource);
     metricsHelper.assertCounter("checkMutatePassedCount", 1, serverSource);
+
+    t.close();
   }
 
   @Test
@@ -267,6 +275,8 @@ public class TestRegionServerMetrics {
 
     metricsRegionServer.getRegionServerWrapper().forceRecompute();
     metricsHelper.assertCounter("incrementNumOps", 13, serverSource);
+
+    t.close();
   }
 
   @Test
@@ -297,5 +307,7 @@ public class TestRegionServerMetrics {
 
     metricsRegionServer.getRegionServerWrapper().forceRecompute();
     metricsHelper.assertCounter("appendNumOps", 73, serverSource);
+
+    t.close();
   }
 }