hadoop-common-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cdoug...@apache.org
Subject svn commit: r666513 - in /hadoop/core/branches/branch-0.18: CHANGES.txt src/core/org/apache/hadoop/fs/FileSystem.java
Date Wed, 11 Jun 2008 03:06:52 GMT
Author: cdouglas
Date: Tue Jun 10 20:06:52 2008
New Revision: 666513

URL: http://svn.apache.org/viewvc?rev=666513&view=rev
Log:
HADOOP-3467. Improve documentation for FileSystem::deleteOnExit.
Contributed by Tsz Wo (Nicholas), SZE.


Modified:
    hadoop/core/branches/branch-0.18/CHANGES.txt
    hadoop/core/branches/branch-0.18/src/core/org/apache/hadoop/fs/FileSystem.java

Modified: hadoop/core/branches/branch-0.18/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.18/CHANGES.txt?rev=666513&r1=666512&r2=666513&view=diff
==============================================================================
--- hadoop/core/branches/branch-0.18/CHANGES.txt (original)
+++ hadoop/core/branches/branch-0.18/CHANGES.txt Tue Jun 10 20:06:52 2008
@@ -276,6 +276,9 @@
     HADOOP-3492. Add forrest documentation for user archives.
     (Mahadev Konar via hairong)
 
+    HADOOP-3467. Improve documentation for FileSystem::deleteOnExit.
+    (Tsz Wo (Nicholas), SZE via cdouglas)
+
   OPTIMIZATIONS
 
     HADOOP-3274. The default constructor of BytesWritable creates empty 

Modified: hadoop/core/branches/branch-0.18/src/core/org/apache/hadoop/fs/FileSystem.java
URL: http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.18/src/core/org/apache/hadoop/fs/FileSystem.java?rev=666513&r1=666512&r2=666513&view=diff
==============================================================================
--- hadoop/core/branches/branch-0.18/src/core/org/apache/hadoop/fs/FileSystem.java (original)
+++ hadoop/core/branches/branch-0.18/src/core/org/apache/hadoop/fs/FileSystem.java Tue Jun
10 20:06:52 2008
@@ -613,9 +613,15 @@
    */
   public abstract boolean delete(Path f, boolean recursive) throws IOException;
 
-  /** Mark a path to be deleted when FileSystem is closed or JVM exits.
-   *  The path has to exist in the file system.
+  /**
+   * Mark a path to be deleted when FileSystem is closed.
+   * When the JVM shuts down,
+   * all FileSystem objects will be closed automatically.
+   * Then,
+   * the marked path will be deleted as a result of closing the FileSystem.
    *
+   * The path has to exist in the file system.
+   * 
    * @param f the path to delete.
    * @return  true if deleteOnExit is successful, otherwise false.
    * @throws IOException
@@ -630,9 +636,10 @@
     return true;
   }
 
-  // Delete all files that were marked as delete-on-exit. This recursively
-  // deletes all files in the specified paths.
-  //
+  /**
+   * Delete all files that were marked as delete-on-exit. This recursively
+   * deletes all files in the specified paths.
+   */
   protected void processDeleteOnExit() {
     synchronized (deleteOnExit) {
       for (Iterator<Path> iter = deleteOnExit.iterator(); iter.hasNext();) {



Mime
View raw message