HDFS-4144. Create test for all snapshot-related metrics. Contributed by Jing Zhao.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/HDFS-2802@1455805 13f79535-47bb-0310-9956-ffa450edef68

hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-2802.txt

@@ -195,3 +195,6 @@ Branch-2802 Snapshot (Unreleased)
 
   HDFS-4563. Update namespace/diskspace usage after deleting snapshots.
   (Jing Zhao via szetszwo)
+
+  HDFS-4144. Create test for all snapshot-related metrics.
+  (Jing Zhao via suresh)

hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/snapshot/INodeDirectorySnapshottable.java

@@ -324,8 +324,8 @@ Snapshot removeSnapshot(String snapshotName,
         Quota.Counts counts = cleanSubtree(snapshot, prior, collectedBlocks);
         INodeDirectory parent = getParent();
         if (parent != null) {
-          parent.addSpaceConsumed(counts.get(Quota.NAMESPACE),
+          parent.addSpaceConsumed(-counts.get(Quota.NAMESPACE),
-              counts.get(Quota.DISKSPACE));
+              -counts.get(Quota.DISKSPACE));
         }
       } catch(QuotaExceededException e) {
         LOG.error("BUG: removeSnapshot increases namespace usage.", e);

hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/snapshot/SnapshotManager.java

@@ -253,6 +253,8 @@ public void removeSnapshottableDirs(
       List<INodeDirectorySnapshottable> toRemoveList) {
     if (toRemoveList != null) {
       this.snapshottables.removeAll(toRemoveList);
+      // modify the numSnapshottableDirs metrics
+      numSnapshottableDirs.addAndGet(-toRemoveList.size());
     }
   }
   

hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/snapshot/TestSnapshot.java

@@ -19,8 +19,10 @@
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
 
 import java.io.File;
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.EnumSet;
@@ -45,7 +47,7 @@
 import org.apache.hadoop.hdfs.server.namenode.FSNamesystem;
 import org.apache.hadoop.hdfs.server.namenode.INode;
 import org.apache.hadoop.hdfs.server.namenode.snapshot.SnapshotTestHelper.TestDirectoryTree;
-import org.apache.hadoop.ipc.RemoteException;
+import org.apache.hadoop.test.GenericTestUtils;
 import org.apache.hadoop.util.Time;
 import org.apache.log4j.Level;
 import org.junit.After;
@@ -266,7 +268,7 @@ private void runTestSnapshot() throws Exception {
    * A simple test that updates a sub-directory of a snapshottable directory
    * with snapshots
    */
-  @Test (timeout=300000)
+  @Test (timeout=60000)
   public void testUpdateDirectory() throws Exception {
     Path dir = new Path("/dir");
     Path sub = new Path(dir, "sub");
@@ -288,11 +290,8 @@ public void testUpdateDirectory() throws Exception {
   
   /**
    * Creating snapshots for a directory that is not snapshottable must fail.
-   * 
-   * TODO: Listing/Deleting snapshots for a directory that is not snapshottable
-   * should also fail.
    */
-  @Test (timeout=300000)
+  @Test (timeout=60000)
   public void testSnapshottableDirectory() throws Exception {
     Path dir = new Path("/TestSnapshot/sub");
     Path file0 = new Path(dir, "file0");
@@ -300,10 +299,29 @@ public void testSnapshottableDirectory() throws Exception {
     DFSTestUtil.createFile(hdfs, file0, BLOCKSIZE, REPLICATION, seed);
     DFSTestUtil.createFile(hdfs, file1, BLOCKSIZE, REPLICATION, seed);
 
-    exception.expect(RemoteException.class);
+    try {
-    exception.expectMessage("Directory is not a snapshottable directory: "
+      hdfs.createSnapshot(dir, "s1");
-        + dir.toString());
+      fail("Exception expected: " + dir + " is not snapshottable");
-    hdfs.createSnapshot(dir, "s1");
+    } catch (IOException e) {
+      GenericTestUtils.assertExceptionContains(
+          "Directory is not a snapshottable directory: " + dir, e);
+    }
+
+    try {
+      hdfs.deleteSnapshot(dir, "s1");
+      fail("Exception expected: " + dir + " is not a snapshottale dir");
+    } catch (Exception e) {
+      GenericTestUtils.assertExceptionContains(
+          "Directory is not a snapshottable directory: " + dir, e);
+    }
+
+    try {
+      hdfs.renameSnapshot(dir, "s1", "s2");
+      fail("Exception expected: " + dir + " is not a snapshottale dir");
+    } catch (Exception e) {
+      GenericTestUtils.assertExceptionContains(
+          "Directory is not a snapshottable directory: " + dir, e);
+    }
   }
 
   /**

hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/snapshot/TestSnapshotDeletion.java

@@ -697,6 +697,7 @@ public void testRenameSnapshotDiff() throws Exception {
     
     // make changes on sub
     hdfs.delete(subFile1, true);
+    checkQuotaUsageComputation(new Path("/"), 16, BLOCKSIZE * 11);
     checkQuotaUsageComputation(dir, 15, BLOCKSIZE * 11);
     checkQuotaUsageComputation(sub, 13, BLOCKSIZE * 11);
     
@@ -717,6 +718,7 @@ public void testRenameSnapshotDiff() throws Exception {
     
     // delete snapshot s2
     hdfs.deleteSnapshot(dir, "s2");
+    checkQuotaUsageComputation(new Path("/"), 14, BLOCKSIZE * 11);
     checkQuotaUsageComputation(dir, 13, BLOCKSIZE * 11);
     checkQuotaUsageComputation(sub, 12, BLOCKSIZE * 11);