svn commit: r620674 - in /hadoop/core/branches/branch-0.16: CHANGES.txt src/java/org/apache/hadoop/fs/ChecksumFileSystem.java

[cdouglas]

Author: cdouglas
Date: Mon Feb 11 16:15:34 2008
New Revision: 620674

URL: http://svn.apache.org/viewvc?rev=620674&view=rev
Log:
HADOOP-2754. Filter out .crc files from local file system listing.
Contributed by Hairong Kuang


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

Modified: hadoop/core/branches/branch-0.16/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.16/CHANGES.txt?rev=620674&r1=620673&r2=620674&view=diff
==============================================================================
--- hadoop/core/branches/branch-0.16/CHANGES.txt (original)
+++ hadoop/core/branches/branch-0.16/CHANGES.txt Mon Feb 11 16:15:34 2008
@@ -25,6 +25,9 @@
                HADOOP-2716. Superuser privileges for the Balancer.
                (Tsz Wo (Nicholas), SZE via shv)
 
+    HADOOP-2754. Filter out .crc files from local file system listing.
+    (Hairong Kuang via cdouglas)
+
 Release 0.16.0 - 2008-02-07
 
   INCOMPATIBLE CHANGES

Modified: 
hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java?rev=620674&r1=620673&r2=620674&view=diff
==============================================================================
--- 
hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java
 (original)
+++ 
hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java
 Mon Feb 11 16:15:34 2008
@@ -425,18 +425,32 @@
   }
 
   final private static PathFilter DEFAULT_FILTER = new PathFilter() {
-      public boolean accept(Path file) {
-        return !isChecksumFile(file);
-      }
-    };
+    public boolean accept(Path file) {
+      return !isChecksumFile(file);
+    }
+  };
 
+  /**
+   * List the statuses of the files/directories in the given path if the path 
is
+   * a directory.
+   * 
+   * @param f
+   *          given path
+   * @return the statuses of the files/directories in the given patch
+   * @throws IOException
+   */
+  @Override
+  public FileStatus[] listStatus(Path f) throws IOException {
+    return fs.listStatus(f, DEFAULT_FILTER);
+  }
+  
   /** 
    * Filter raw files in the given pathes using the default checksum filter. 
    * @param files a list of paths
    * @return a list of files under the source paths
    * @exception IOException
    */
-  @Override
+  @Deprecated
   public Path[] listPaths(Path[] files) throws IOException {
     return fs.listPaths(files, DEFAULT_FILTER);
   }
@@ -447,6 +461,7 @@
    * @return a list of files under the source path
    * @exception IOException
    */
+  @Deprecated
   public Path[] listPaths(Path f) throws IOException {
     return fs.listPaths(f, DEFAULT_FILTER);
   }