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 zh...@apache.org on 2015/02/06 22:47:15 UTC
[43/50] [abbrv] hadoop git commit: HDFS-7741. Remove unnecessary
synchronized in FSDataInputStream and HdfsDataInputStream. (yliu)
HDFS-7741. Remove unnecessary synchronized in FSDataInputStream and HdfsDataInputStream. (yliu)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/aad02110
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/aad02110
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/aad02110
Branch: refs/heads/HDFS-EC
Commit: aad021102dc45f9dea4ae00527ee25e09e4e9991
Parents: e684e78
Author: yliu <yl...@apache.org>
Authored: Fri Feb 6 06:31:03 2015 +0800
Committer: Zhe Zhang <zh...@apache.org>
Committed: Fri Feb 6 13:45:52 2015 -0800
----------------------------------------------------------------------
.../src/main/java/org/apache/hadoop/fs/FSDataInputStream.java | 2 +-
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 3 +++
.../java/org/apache/hadoop/hdfs/client/HdfsDataInputStream.java | 2 +-
3 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/aad02110/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FSDataInputStream.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FSDataInputStream.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FSDataInputStream.java
index c8609d4..6d39d1e 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FSDataInputStream.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FSDataInputStream.java
@@ -58,7 +58,7 @@ public class FSDataInputStream extends DataInputStream
* @param desired offset to seek to
*/
@Override
- public synchronized void seek(long desired) throws IOException {
+ public void seek(long desired) throws IOException {
((Seekable)in).seek(desired);
}
http://git-wip-us.apache.org/repos/asf/hadoop/blob/aad02110/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
index 756b70f..78aa992 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -851,6 +851,9 @@ Release 2.7.0 - UNRELEASED
HDFS-7698. Fix locking on HDFS read statistics and add a method for
clearing them. (Colin P. McCabe via yliu)
+ HDFS-7741. Remove unnecessary synchronized in FSDataInputStream and
+ HdfsDataInputStream. (yliu)
+
Release 2.6.1 - UNRELEASED
INCOMPATIBLE CHANGES
http://git-wip-us.apache.org/repos/asf/hadoop/blob/aad02110/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/client/HdfsDataInputStream.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/client/HdfsDataInputStream.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/client/HdfsDataInputStream.java
index 72c57a8..e8ac686 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/client/HdfsDataInputStream.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/client/HdfsDataInputStream.java
@@ -83,7 +83,7 @@ public class HdfsDataInputStream extends FSDataInputStream {
/**
* Get the collection of blocks that has already been located.
*/
- public synchronized List<LocatedBlock> getAllBlocks() throws IOException {
+ public List<LocatedBlock> getAllBlocks() throws IOException {
return getDFSInputStream().getAllBlocks();
}