You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by cu...@apache.org on 2008/06/02 20:38:08 UTC

svn commit: r662509 - in /hadoop/core/trunk: CHANGES.txt lib/kfs-0.1.3.jar lib/kfs-0.1.jar src/java/org/apache/hadoop/fs/kfs/KFSImpl.java

Author: cutting
Date: Mon Jun  2 11:38:07 2008
New Revision: 662509

URL: http://svn.apache.org/viewvc?rev=662509&view=rev
Log:
HADOOP-3410.  Fix KFS implementation to return correct file modification time.  Contributed by Sriram Rao.

Added:
    hadoop/core/trunk/lib/kfs-0.1.3.jar   (with props)
Removed:
    hadoop/core/trunk/lib/kfs-0.1.jar
Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/java/org/apache/hadoop/fs/kfs/KFSImpl.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=662509&r1=662508&r2=662509&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Mon Jun  2 11:38:07 2008
@@ -368,6 +368,9 @@
     HADOOP-3477. Fix build to not package contrib/*/bin twice in
     distributions.  (Adam Heath via cutting)
 
+    HADOOP-3410. Fix KFS implemenation to return correct file
+    modification time.  (Sriram Rao via cutting)
+
 
 Release 0.17.0 - 2008-05-18
 

Added: hadoop/core/trunk/lib/kfs-0.1.3.jar
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/lib/kfs-0.1.3.jar?rev=662509&view=auto
==============================================================================
Binary file - no diff available.

Propchange: hadoop/core/trunk/lib/kfs-0.1.3.jar
------------------------------------------------------------------------------
    svn:mime-type = application/octet-stream

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/fs/kfs/KFSImpl.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/fs/kfs/KFSImpl.java?rev=662509&r1=662508&r2=662509&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/fs/kfs/KFSImpl.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/fs/kfs/KFSImpl.java Mon Jun  2 11:38:07 2008
@@ -92,10 +92,7 @@
     }
 
     public long getModificationTime(String path) throws IOException {
-        // Supporting this API requires changes to the Java-side of
-        // the KFS client API.  For now, return 0; in the next rev of
-        // KFS, we'll update the Java API.
-        return 0;
+        return kfsAccess.kfs_getModificationTime(path);
     }
 
     public FSDataOutputStream create(String path, short replication, int bufferSize) throws IOException {