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

Zhijie Shen commented on YARN-2317:
-----------------------------------

[~gtCarrera9], thanks for your update and summary of the change. Here're some 
more comments.

1. Change it to a question?
{code}
My container is being killed by the node manager
{code}

2.  Please search through the whole doc for "Resource Manager", "Node Manager" 
and "Application Master". How about removing " " between the two words, turning 
them into special terminologies of YARN?

3. simpler implementations -> I think you mean simpler usage?
{code}
(subjectively) simpler implementations
{code}

4. The test related code may not be included?
{code}
  // add the runtime classpath needed for tests to work
  if (conf.getBoolean(YarnConfiguration.IS_MINI_YARN_CLUSTER, false)) {
    classPathEnv.append(':');
    classPathEnv.append(System.getProperty("java.class.path"));
  }
{code}

5. The front of the code block under this question seems to be bigger than that 
of others.
{code}
How can I distribute my application's jars to all of the nodes in the YARN 
cluster that need it?
{code}

> Update documentation about how to write YARN applications
> ---------------------------------------------------------
>
>                 Key: YARN-2317
>                 URL: https://issues.apache.org/jira/browse/YARN-2317
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: documentation
>            Reporter: Li Lu
>            Assignee: Li Lu
>             Fix For: 2.6.0
>
>         Attachments: YARN-2317-071714.patch, YARN-2317-073014-1.patch, 
> YARN-2317-073014.patch, YARN-2317-081114.patch
>
>
> Some information in WritingYarnApplications webpage is out-dated. Need some 
> refresh work on this document to reflect the most recent changes in YARN 
> APIs. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to