svn commit: r1609276 - in /hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native: jni/jnifs.c ndfs/ndfs.c

[cmccabe]

Author: cmccabe
Date: Wed Jul  9 19:29:26 2014
New Revision: 1609276

URL: http://svn.apache.org/r1609276
Log:
HADOOP-10805. ndfs hdfsDelete should check the return boolean (cmccabe)

Modified:
    
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/jni/jnifs.c
    
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/ndfs/ndfs.c

Modified: 
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/jni/jnifs.c
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/jni/jnifs.c?rev=1609276&r1=1609275&r2=1609276&view=diff
==============================================================================
--- 
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/jni/jnifs.c
 (original)
+++ 
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/jni/jnifs.c
 Wed Jul  9 19:29:26 2014
@@ -1382,7 +1382,7 @@ static int jni_unlink(hdfsFS bfs, const 
         return -1;
     }
     if (!jVal.z) {
-        errno = EIO;
+        errno = ENOENT;
         return -1;
     }
     return 0;

Modified: 
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/ndfs/ndfs.c
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/ndfs/ndfs.c?rev=1609276&r1=1609275&r2=1609276&view=diff
==============================================================================
--- 
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/ndfs/ndfs.c
 (original)
+++ 
hadoop/common/branches/HADOOP-10388/hadoop-native-core/src/main/native/ndfs/ndfs.c
 Wed Jul  9 19:29:26 2014
@@ -593,6 +593,11 @@ static int ndfs_unlink(struct hdfs_inter
     if (err) {
         goto done;
     }
+    if (resp->result == 0) {
+        err = hadoop_lerr_alloc(ENOENT, "ndfs_unlink(%s, recursive=%d): "
+                    "deletion failed on the server", uri, recursive);
+        goto done;
+    }
 
 done:
     free(path);