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 iw...@apache.org on 2020/08/18 04:58:58 UTC

[hadoop] branch branch-2.10 updated: HADOOP-17202. Fix findbugs warnings in hadoop-tools on branch-2.10. (#2214)

This is an automated email from the ASF dual-hosted git repository.

iwasakims pushed a commit to branch branch-2.10
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/branch-2.10 by this push:
     new a9f82a1  HADOOP-17202. Fix findbugs warnings in hadoop-tools on branch-2.10. (#2214)
a9f82a1 is described below

commit a9f82a11df768d80c5a05761e74750fe4f291b70
Author: Masatake Iwasaki <iw...@apache.org>
AuthorDate: Tue Aug 18 13:58:35 2020 +0900

    HADOOP-17202. Fix findbugs warnings in hadoop-tools on branch-2.10. (#2214)
---
 .../org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java     | 6 +-----
 .../java/org/apache/hadoop/tools/rumen/JsonObjectMapperWriter.java  | 1 -
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java b/hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java
index cb12645..c3dfb11 100644
--- a/hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java
+++ b/hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java
@@ -894,13 +894,9 @@ public class AzureBlobFileSystemStore {
           + " used is not namespace enabled");
       return;
     }
-    //try (AbfsPerfInfo perfInfo = startTracking("access", "checkAccess")) {
       String relativePath =
           AbfsHttpConstants.FORWARD_SLASH + getRelativePath(path, true);
-      final AbfsRestOperation op = this.client
-          .checkAccess(relativePath, mode.SYMBOL);
-    // perfInfo.registerResult(op.getResult()).registerSuccess(true);
-    //}
+      this.client.checkAccess(relativePath, mode.SYMBOL);
   }
 
   public boolean isAtomicRenameKey(String key) {
diff --git a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JsonObjectMapperWriter.java b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JsonObjectMapperWriter.java
index 47bfee0..212ed8b 100644
--- a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JsonObjectMapperWriter.java
+++ b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JsonObjectMapperWriter.java
@@ -56,7 +56,6 @@ public class JsonObjectMapperWriter<T> implements Closeable {
     // register the module with the object-mapper
     mapper.registerModule(module);
 
-    mapper.getJsonFactory();
     writer = mapper.getJsonFactory().createJsonGenerator(
         output, JsonEncoding.UTF8);
     if (prettyPrint) {


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org