[ https://issues.apache.org/jira/browse/YARN-2581?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14145460#comment-14145460 ]
Zhijie Shen commented on YARN-2581: ----------------------------------- Basically the approach in this patch is to embed LogAggregationContext into ContainerTokenIdentifier, which will be passed to AM when allocating containers. Afterwards, AM can use it to start a container on NM. The merit is that LogAggregationContext is automatically forwarded to NM without app user's involvement. The approach makes sense to me. 1. The old constructor call the new one? Same for ApplicationInitEvent and LogHandlerAppStartedEvent. {code} public ContainerTokenIdentifier(ContainerId containerID, String hostName, String appSubmitter, Resource r, long expiryTimeStamp, @@ -74,6 +78,22 @@ public ContainerTokenIdentifier(ContainerId containerID, this.creationTime = creationTime; } {code} 2. Exchange two params? First should be expected, and the second should be actual? {code} + assertEquals(recovered.getRollingIntervalSeconds(), + logAggregationContext.getRollingIntervalSeconds()); {code} 3. No need to remember appSubmissionContext, but remember LogAggregationContext directly? {code} + private ApplicationSubmissionContext appSubmissionContext = null; {code} > NMs need to find a way to get LogAggregationContext > --------------------------------------------------- > > Key: YARN-2581 > URL: https://issues.apache.org/jira/browse/YARN-2581 > Project: Hadoop YARN > Issue Type: Sub-task > Components: nodemanager, resourcemanager > Reporter: Xuan Gong > Assignee: Xuan Gong > Attachments: YARN-2581.1.patch, YARN-2581.2.patch > > > After YARN-2569, we have LogAggregationContext for application in > ApplicationSubmissionContext. NMs need to find a way to get this information. > We have this requirement: For all containers in the same application should > honor the same LogAggregationContext. -- This message was sent by Atlassian JIRA (v6.3.4#6332)