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 ae...@apache.org on 2017/04/04 19:56:53 UTC
[29/50] [abbrv] hadoop git commit: HDFS-11592. Closing a file has a
wasteful preconditions in NameNode. Contributed by Eric Badger
HDFS-11592. Closing a file has a wasteful preconditions in NameNode. Contributed by Eric Badger
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/e5f0622a
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/e5f0622a
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/e5f0622a
Branch: refs/heads/HDFS-7240
Commit: e5f0622a6f40706d360d45200c8f259c79046438
Parents: b58777a
Author: Mingliang Liu <li...@apache.org>
Authored: Thu Mar 30 15:44:06 2017 -0700
Committer: Mingliang Liu <li...@apache.org>
Committed: Thu Mar 30 15:44:21 2017 -0700
----------------------------------------------------------------------
.../org/apache/hadoop/hdfs/server/namenode/INodeFile.java | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/e5f0622a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INodeFile.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INodeFile.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INodeFile.java
index 2b0e0ad..3da6aa7 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INodeFile.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INodeFile.java
@@ -328,9 +328,11 @@ public class INodeFile extends INodeWithAdditionalFields
for (int i = 0; i < blocks.length; i++) {
final String err = checkBlockComplete(blocks, i, numCommittedAllowed,
minReplication);
- Preconditions.checkState(err == null,
- "Unexpected block state: %s, file=%s (%s), blocks=%s (i=%s)",
- err, this, getClass().getSimpleName(), Arrays.asList(blocks), i);
+ if(err != null) {
+ throw new IllegalStateException(String.format("Unexpected block state: " +
+ "%s, file=%s (%s), blocks=%s (i=%s)", err, this,
+ getClass().getSimpleName(), Arrays.asList(blocks), i));
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org