Minor cleanup.

Alluxio · Nov 23, 2016 · 6abb706 · 6abb706
1 parent 43596bd
commit 6abb706
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/core/server/src/main/java/alluxio/proxy/FileSystemClientRestServiceHandler.java b/core/server/src/main/java/alluxio/proxy/FileSystemClientRestServiceHandler.java
@@ -277,22 +277,21 @@ public URIStatus call() throws Exception {
   /**
    * @summary get the file descriptors for a path
    * @param path the file path
-   * @param loadMetadataType the {@link LoadMetadataType}. It overrides loadDirectChildren if it
+   * @param loadMetadataType the load metadata type
-   *        is set.
    * @return the response object
    */
   @GET
   @Path(LIST_STATUS)
   @ReturnType("java.util.List<alluxio.client.file.URIStatus>")
   public Response listStatus(@QueryParam("path") final String path,
-      @QueryParam("loadMetadataType") final String loadMetadataType) {
+      @QueryParam("loadMetadataType") final LoadMetadataType loadMetadataType) {
     return RestUtils.call(new RestUtils.RestCallable<List<URIStatus>>() {
       @Override
       public List<URIStatus> call() throws Exception {
         Preconditions.checkNotNull(path, "required 'path' parameter is missing");
         ListStatusOptions listStatusOptions = ListStatusOptions.defaults();
-        if (!loadMetadataType.isEmpty()) {
+        if (loadMetadataType != null) {
-          listStatusOptions.setLoadMetadataType(LoadMetadataType.valueOf(loadMetadataType));
+          listStatusOptions.setLoadMetadataType(loadMetadataType);
         }
         return mFileSystem.listStatus(new AlluxioURI(path), listStatusOptions);
       }