hadoop-common-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dran...@apache.org
Subject [17/50] [abbrv] hadoop git commit: HADOOP-13559. Remove close() within try-with-resources in ChecksumFileSystem/ChecksumFs classes. (Contributed by Aaron Fabbri)
Date Tue, 06 Sep 2016 08:47:55 GMT
HADOOP-13559. Remove close() within try-with-resources in ChecksumFileSystem/ChecksumFs classes.
(Contributed by Aaron Fabbri)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/6fcb04c1
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/6fcb04c1
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/6fcb04c1

Branch: refs/heads/HADOOP-12756
Commit: 6fcb04c1780ac3dca5b986f1bcd558fffccb3eb9
Parents: 69f7277
Author: Mingliang Liu <liuml07@apache.org>
Authored: Mon Aug 29 13:04:28 2016 -0700
Committer: Mingliang Liu <liuml07@apache.org>
Committed: Mon Aug 29 13:04:28 2016 -0700

----------------------------------------------------------------------
 .../src/main/java/org/apache/hadoop/fs/ChecksumFileSystem.java      | 1 -
 .../src/main/java/org/apache/hadoop/fs/ChecksumFs.java              | 1 -
 2 files changed, 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/6fcb04c1/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFileSystem.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFileSystem.java
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFileSystem.java
index e4c0b33..e0ce327 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFileSystem.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFileSystem.java
@@ -197,7 +197,6 @@ public abstract class ChecksumFileSystem extends FilterFileSystem {
                new ChecksumFSInputChecker(fs, file)) {
         checker.seek(position);
         nread = checker.read(b, off, len);
-        checker.close();
       }
       return nread;
     }

http://git-wip-us.apache.org/repos/asf/hadoop/blob/6fcb04c1/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFs.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFs.java
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFs.java
index 6e98db5..3972033 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFs.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/ChecksumFs.java
@@ -179,7 +179,6 @@ public abstract class ChecksumFs extends FilterFs {
                new ChecksumFSInputChecker(fs, file)) {
         checker.seek(position);
         nread = checker.read(b, off, len);
-        checker.close();
       }
       return nread;
     }


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


Mime
View raw message