[
https://issues.apache.org/jira/browse/HADOOP-14411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16007438#comment-16007438
]
Hadoop QA commented on HADOOP-14411:
------------------------------------
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m
17s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 16m
27s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m
6s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m
2s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m
0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m
27s{color} | {color:green} The patch does not generate ASF License warnings.
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 19m 45s{color} |
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Image:yetus/hadoop:14b5c93 |
| JIRA Issue | HADOOP-14411 |
| JIRA Patch URL |
https://issues.apache.org/jira/secure/attachment/12867681/HADOOP-14411.002.patch
|
| Optional Tests | asflicense mvnsite |
| uname | Linux d7fcdc9c5629 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20
09:57:27 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh
|
| git revision | trunk / 0d5c8ed |
| modules | C: hadoop-hdfs-project/hadoop-hdfs U:
hadoop-hdfs-project/hadoop-hdfs |
| Console output |
https://builds.apache.org/job/PreCommit-HADOOP-Build/12298/console |
| Powered by | Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org |
This message was automatically generated.
> HDFS architecture documentation describes outdated placement policy
> -------------------------------------------------------------------
>
> Key: HADOOP-14411
> URL: https://issues.apache.org/jira/browse/HADOOP-14411
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: dud
> Assignee: Chen Liang
> Priority: Minor
> Labels: documentation
> Attachments: HADOOP-14411.001.patch, HADOOP-14411.002.patch
>
>
> Hello
> I've noticed inconsistencies in the default block placement policy between
> what one can read on several websites and what is stated in the [official
> documentation|https://git-wip-us.apache.org/repos/asf?p=hadoop.git;a=blob;f=hadoop-hdfs-project/hadoop-hdfs/src/site/markdown/HdfsDesign.md;h=bda53a9c77b41308eef3abf5b478158b2ea03be0;hb=HEAD#l105].
> After digging, I've found out that the proper default placement policy is :
> {code}
> * The replica placement strategy is that if the writer is on a datanode,
> * the 1st replica is placed on the local machine,
> * otherwise a random datanode. The 2nd replica is placed on a datanode
> * that is on a different rack. The 3rd replica is placed on a datanode
> * which is on a different node of the rack as the second replica.
> {code}
> [source|https://git-wip-us.apache.org/repos/asf?p=hadoop.git;a=blob;f=hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockPlacementPolicyDefault.java#l45]
> HADOOP-5734 has been opened regarding this mistake several years ago and has
> eventually been merged but unfortunately this fix has been overwritten by
> [~tucu00] in [SVN
> commit|https://svn.apache.org/viewvc?view=revision&revision=1425527].
> dud
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
