Repository: hadoop Updated Branches: refs/heads/branch-2 1405181d0 -> b50af4e04
HDFS-7284. Add more debug info to BlockInfoUnderConstruction#setGenerationStampAndVerifyReplicas. (Wei-Chiu Chuang via Yongjun Zhang) (cherry picked from commit 5e718de522328d1112ad38063596c204aa43f539) Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/b50af4e0 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/b50af4e0 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/b50af4e0 Branch: refs/heads/branch-2 Commit: b50af4e04adfd3d106dce98911a660d58da84bba Parents: 1405181 Author: Yongjun Zhang <[email protected]> Authored: Mon Oct 26 13:16:11 2015 -0700 Committer: Yongjun Zhang <[email protected]> Committed: Mon Oct 26 13:36:25 2015 -0700 ---------------------------------------------------------------------- .../java/org/apache/hadoop/hdfs/protocol/Block.java | 13 ++++++++++++- hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 4 ++++ .../hadoop/hdfs/server/blockmanagement/BlockInfo.java | 4 ++-- 3 files changed, 18 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/b50af4e0/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/Block.java ---------------------------------------------------------------------- diff --git a/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/Block.java b/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/Block.java index 710897e..2b139b2 100644 --- a/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/Block.java +++ b/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/Block.java @@ -153,10 +153,21 @@ public class Block implements Writable, Comparable<Block> { } /** + * A helper method to output the string representation of the Block portion of + * a derived class' instance. + * + * @param b the target object + * @return the string representation of the block + */ + public static String toString(final Block b) { + return b.getBlockName() + "_" + b.getGenerationStamp(); + } + + /** */ @Override public String toString() { - return getBlockName() + "_" + getGenerationStamp(); + return toString(this); } public void appendStringTo(StringBuilder sb) { http://git-wip-us.apache.org/repos/asf/hadoop/blob/b50af4e0/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 7d1cc17..ebd017d 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt @@ -733,6 +733,10 @@ Release 2.8.0 - UNRELEASED HDFS-4015. Safemode should count and report orphaned blocks. (Anu Engineer via Arpit Agarwal) + HDFS-7284. Add more debug info to + BlockInfoUnderConstruction#setGenerationStampAndVerifyReplicas. + (Wei-Chiu Chuang via Yongjun Zhang) + OPTIMIZATIONS HDFS-8026. Trace FSOutputSummer#writeChecksumChunks rather than http://git-wip-us.apache.org/repos/asf/hadoop/blob/b50af4e0/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockInfo.java ---------------------------------------------------------------------- diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockInfo.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockInfo.java index db5d42a..ecb584f 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockInfo.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockInfo.java @@ -391,8 +391,8 @@ public abstract class BlockInfo extends Block List<ReplicaUnderConstruction> staleReplicas = uc.getStaleReplicas(genStamp); for (ReplicaUnderConstruction r : staleReplicas) { r.getExpectedStorageLocation().removeBlock(this); - NameNode.blockStateChangeLog.debug("BLOCK* Removing stale replica " - + "from location: {}", r.getExpectedStorageLocation()); + NameNode.blockStateChangeLog.debug("BLOCK* Removing stale replica {}" + + " of {}", r, Block.toString(r)); } }
