[ 
https://issues.apache.org/jira/browse/HDFS-12570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16192915#comment-16192915
 ] 

Surendra Singh Lilhore commented on HDFS-12570:
-----------------------------------------------

Thanks [~rakeshr] for patch...

I looked at the patch. Following are my comment in my first look.

# In 
{{BlockStorageMovementAttemptedItems#blockStorageMovementReportedItemsCheck()}},
 if {{attemptedItemInfo.allBlockLocsAttemptedToSatisfy}} is false then we 
should add {{ItemInfo}} back to the {{blockStorageMovementNeeded}} queue.
# FSNamesystem#handleHeartbeat() : Correct the log message, now are not using 
"co-ordinator datanode".
{code}
          if (LOG.isDebugEnabled()) {
            LOG.debug(
                "Storage policy satisfier is not running. So, ignoring block "
                    + "storage movement results sent by co-ordinator datanode");
          }
{code}
# Can we try to use IBR instead of adding new parameter 
{{BlocksStorageMoveAttemptFinished}} in {{NameNodeRpcServer#sendHeartbeat()}} ?.

> [SPS]: Refactor Co-ordinator datanode logic to track the block storage 
> movements
> --------------------------------------------------------------------------------
>
>                 Key: HDFS-12570
>                 URL: https://issues.apache.org/jira/browse/HDFS-12570
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: datanode, namenode
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>         Attachments: HDFS-12570-HDFS-10285-00.patch
>
>
> This task is to refactor the C-DN block storage movements. Basically, the 
> idea is to move the scheduling and tracking logic to Namenode rather than at 
> the special C-DN. Please refer the discussion with [~andrew.wang] to 
> understand the [background and the necessity of 
> refactoring|https://issues.apache.org/jira/browse/HDFS-10285?focusedCommentId=16141060&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16141060].



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to