You are viewing a plain text version of this content. The canonical link for it is here.

Posted to common-dev@hadoop.apache.org by "Konstantin Shvachko (JIRA)" <ji...@apache.org> on 2007/01/27 01:25:49 UTC

[jira] Created: (HADOOP-944) Redundant field: DatanodeDescriptor.isAlive

Redundant field: DatanodeDescriptor.isAlive
-------------------------------------------

                 Key: HADOOP-944
                 URL: https://issues.apache.org/jira/browse/HADOOP-944
             Project: Hadoop
          Issue Type: Improvement
            Reporter: Konstantin Shvachko


The lastUpdate field should be used for determining whether the data-node is dead or alive.
The DatanodeDescriptor should have isAlive() method rather than the field.
Synchronizing different fields having the same meaning often leads to mistakes.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Updated: (HADOOP-944) Redundant field: DatanodeDescriptor.isAlive

Posted by "dhruba borthakur (JIRA)" <ji...@apache.org>.

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

dhruba borthakur updated HADOOP-944:
------------------------------------

    Component/s: dfs

> Redundant field: DatanodeDescriptor.isAlive
> -------------------------------------------
>
>                 Key: HADOOP-944
>                 URL: https://issues.apache.org/jira/browse/HADOOP-944
>             Project: Hadoop
>          Issue Type: Improvement
>          Components: dfs
>            Reporter: Konstantin Shvachko
>
> The lastUpdate field should be used for determining whether the data-node is dead or alive.
> The DatanodeDescriptor should have isAlive() method rather than the field.
> Synchronizing different fields having the same meaning often leads to mistakes.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Commented: (HADOOP-944) Redundant field: DatanodeDescriptor.isAlive

Posted by "dhruba borthakur (JIRA)" <ji...@apache.org>.

    [ https://issues.apache.org/jira/browse/HADOOP-944?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12468351 ] 

dhruba borthakur commented on HADOOP-944:
-----------------------------------------

I agree. 

We also should keep in mind that reducing the number of calls to gettimeofday() is a good thing.

> Redundant field: DatanodeDescriptor.isAlive
> -------------------------------------------
>
>                 Key: HADOOP-944
>                 URL: https://issues.apache.org/jira/browse/HADOOP-944
>             Project: Hadoop
>          Issue Type: Improvement
>            Reporter: Konstantin Shvachko
>
> The lastUpdate field should be used for determining whether the data-node is dead or alive.
> The DatanodeDescriptor should have isAlive() method rather than the field.
> Synchronizing different fields having the same meaning often leads to mistakes.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.