hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lei (Eddy) Xu (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-10808) DiskBalancer does not execute multi-steps plan-redux
Date Wed, 31 Aug 2016 00:31:21 GMT

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

Lei (Eddy) Xu commented on HDFS-10808:

Thanks for the explanation, [~anu]

bq.  The intend of having the setExitFlag was to reduce the complexity of multiple exits from
the loop. 

Wouldn't {{break}} is the simplest way to exit the while loop? 

bq. Fortunately for us, the way we do cancel is by cancelling the executor and not by relying
on this flag. 

It makes me wonder why we have the flag in the first place. And for the comments of the following

      // check if someone told us exit, treat this as an interruption
            // point
            // for the thread, since both getNextBlock and moveBlocAcrossVolume
            // can take some time.
            if (!shouldRun()) {

If {{shouldRun}} is not used for canceling, it is very confused to me because all the {{exitFlag}}
and {{shouldRun()}} are only consumed within {{copyBlocks()}}, and use atomic boolean for
{{shouldRun}} flag.

> DiskBalancer does not execute multi-steps plan-redux
> ----------------------------------------------------
>                 Key: HDFS-10808
>                 URL: https://issues.apache.org/jira/browse/HDFS-10808
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: balancer & mover
>            Reporter: Anu Engineer
>            Assignee: Anu Engineer
>         Attachments: HDFS-10808.001.patch, HDFS-10808.002.patch
> This is to redo of the fix in HDFS-10598

This message was sent by Atlassian JIRA

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

View raw message