refactor to make restapi happy

Alluxio · May 3, 2017 · a50b10a · a50b10a
1 parent 16e1884
commit a50b10a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/core/client/src/main/java/alluxio/client/file/options/DeleteOptions.java b/core/client/src/main/java/alluxio/client/file/options/DeleteOptions.java
@@ -91,11 +91,11 @@ public DeleteOptions setAlluxioOnly(boolean alluxioOnly) {
   }
 
   /**
-   * @param skipCheck whether to skip UFS sync check
+   * @param unchecked whether to skip UFS sync check
    * @return the updated options object
    */
-  public DeleteOptions skipCheck(boolean skipCheck) {
+  public DeleteOptions setUnchecked(boolean unchecked) {
-    mUnchecked = skipCheck;
+    mUnchecked = unchecked;
     return this;
   }
 

diff --git a/core/client/src/test/java/alluxio/client/file/options/DeleteOptionsTest.java b/core/client/src/test/java/alluxio/client/file/options/DeleteOptionsTest.java
@@ -52,7 +52,7 @@ public void fields() {
 
     options.setRecursive(recursive);
     options.setAlluxioOnly(alluxioOnly);
-    options.skipCheck(unchecked);
+    options.setUnchecked(unchecked);
     Assert.assertEquals(recursive, options.isRecursive());
     Assert.assertEquals(alluxioOnly, options.isAlluxioOnly());
     Assert.assertEquals(unchecked, options.isUnchecked());

diff --git a/shell/src/main/java/alluxio/shell/command/RmCommand.java b/shell/src/main/java/alluxio/shell/command/RmCommand.java
@@ -67,7 +67,7 @@ protected void runCommand(AlluxioURI path, CommandLine cl) throws AlluxioExcepti
 
     DeleteOptions options = DeleteOptions.defaults().setRecursive(recursive);
     if (cl.hasOption(REMOVE_UNCHECKED_OPTION_CHAR)) {
-      options.skipCheck(true);
+      options.setUnchecked(true);
     }
     mFileSystem.delete(path, options);
     System.out.println(path + " has been removed");