From d5d1b896450511b516617c58c2bd16fea7db072d Mon Sep 17 00:00:00 2001
From: Tsz-wo Sze <szetszwo@apache.org>
Date: Wed, 12 Jun 2013 05:44:39 +0000
Subject: [PATCH] svn merge -c 1492062 from trunk for HDFS-4819. Update
 Snapshot doc to clarify that nested snapshots are not allowed.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1492064 13f79535-47bb-0310-9956-ffa450edef68
---
 hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt               | 3 +++
 .../hadoop-hdfs/src/site/xdoc/HdfsSnapshots.xml           | 8 ++++----
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
index f3dc4e38772..965bb6984ee 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -860,6 +860,9 @@ Release 2.1.0-beta - UNRELEASED
     HDFS-4873. callGetBlockLocations returns incorrect number of blocks for 
     snapshotted files. (jing9)
 
+    HDFS-4819. Update Snapshot doc to clarify that nested snapshots are not
+    allowed.  (szetszwo)
+
 Release 2.0.5-alpha - 06/06/2013
 
   INCOMPATIBLE CHANGES
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/site/xdoc/HdfsSnapshots.xml b/hadoop-hdfs-project/hadoop-hdfs/src/site/xdoc/HdfsSnapshots.xml
index 3809143da0b..9aecf9cd55a 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/site/xdoc/HdfsSnapshots.xml
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/site/xdoc/HdfsSnapshots.xml
@@ -70,13 +70,13 @@
     the directory can be neither deleted nor renamed
     before all the snapshots are deleted.
   </p>
-<!--
+
   <p>
     Nested snapshottable directories are currently not allowed.
     In other words, a directory cannot be set to snapshottable
-    if one of its ancestors is a snapshottable directory.
+    if one of its ancestors/descendants is a snapshottable directory.
   </p>
--->
+
   </subsection>
 
   <subsection name="Snapshot Paths" id="SnapshotPaths">
@@ -100,7 +100,7 @@
       <source>hdfs dfs -cp /foo/.snapshot/s0/bar /tmp</source></li>
   </ul>
   <p>
-    The name ".snapshot" is now a reserved file name in HDFS
+    <b>Note</b> that the name ".snapshot" is now a reserved file name in HDFS
     so that users cannot create a file/directory with ".snapshot" as the name.
     If ".snapshot" is used in a previous version of HDFS, it must be renamed before upgrade;
     otherwise, upgrade will fail.