You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-dev@hadoop.apache.org by "Xiaoyu Yao (JIRA)" <ji...@apache.org> on 2018/07/25 16:18:00 UTC

[jira] [Resolved] (HDDS-292) Fix ContainerMapping#getMatchingContainerWithPipeline

     [ https://issues.apache.org/jira/browse/HDDS-292?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Xiaoyu Yao resolved HDDS-292.
-----------------------------
    Resolution: Fixed

Based on offline discussion with [~msingh], this will be fixed as part of HDDS-277. 

> Fix ContainerMapping#getMatchingContainerWithPipeline
> -----------------------------------------------------
>
>                 Key: HDDS-292
>                 URL: https://issues.apache.org/jira/browse/HDDS-292
>             Project: Hadoop Distributed Data Store
>          Issue Type: Bug
>            Reporter: Xiaoyu Yao
>            Assignee: Xiaoyu Yao
>            Priority: Major
>             Fix For: 0.2.1
>
>
> The current code does not update the pipeline that is newly allocated 
> {code}
> --- a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerMapping.java
> +++ b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerMapping.java
> @@ -446,7 +446,7 @@ public ContainerWithPipeline getMatchingContainerWithPipeline(final long size,
>          .getPipeline(containerInfo.getPipelineName(),
>              containerInfo.getReplicationType());
>      if (pipeline == null) {
> -      pipelineSelector
> +      pipeline = pipelineSelector
>            .getReplicationPipeline(containerInfo.getReplicationType(),
>                containerInfo.getReplicationFactor());
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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