hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Karthik Kambatla (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-1757) Auxiliary service support for nodemanager recovery
Date Sun, 06 Apr 2014 00:12:15 GMT

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

Karthik Kambatla commented on YARN-1757:

Thanks Jason. Looks mostly good - minor comments and nits. Feel free to ignore nits you don't
agree with. 

# Nit: YarnConfiguration: We might want to add a NM_RECOVERY_PREFIX for all recovery related
# The default recovery-dir should probably be something more specific to nm-recovery - {{/tmp/yarn-nm-recovery}}?

# Nit: Should we add an NMUtils class for static helper methods like isRecoveryEnabled()?
# Nit: Rename variables to stateStore instead of stateStorage - that would go with the conventions
used in RM better, and is shorter :)
    Path stateStorageRoot = null;
    FileSystem stateStorageFs = null;
# Nit: AuxServices#createStorageDir: May be add a comment to say control flow through FileNotFound
is cheaper than explicitly checking if the file exists?
# NameNode should also use something similar, instead of directly creating the directory?
May be, another candidate to move to NMUtils? 
# TestAuxServices: We should check if we have two directories created too?
      final AuxServices aux = new AuxServices();
      Assert.assertEquals(2, aux.getServices().size());

> Auxiliary service support for nodemanager recovery
> --------------------------------------------------
>                 Key: YARN-1757
>                 URL: https://issues.apache.org/jira/browse/YARN-1757
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>    Affects Versions: 2.3.0
>            Reporter: Jason Lowe
>            Assignee: Jason Lowe
>         Attachments: YARN-1757.patch, YARN-1757.patch
> There needs to be a mechanism for communicating to auxiliary services whether nodemanager
recovery is enabled and where they should store their state.

This message was sent by Atlassian JIRA

View raw message