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

Sidharta Seethana commented on YARN-4253:
-----------------------------------------

Pre-Patch failure : unrelated to this JIRA
Tests : There are existing tests in TestLinuxContainerExecutorWithMocks that 
test the part of code that was modified in this patch. There is no change in 
behavior so no additional tests are required.
Release-Audit failure : Not related to this JIRA
Checkstyle failure : the failure is due to formatting changes to a function 
that was not otherwise modified. The length of the function was not changed in 
this patch. 

[~vvasudev] , could you please review this patch?

> Standardize on using PrivilegedOperationExecutor for all invocations of 
> container-executor in LinuxContainerExecutor
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-4253
>                 URL: https://issues.apache.org/jira/browse/YARN-4253
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Sidharta Seethana
>            Assignee: Sidharta Seethana
>         Attachments: YARN-4253.001.patch
>
>
> YARN-3443 introduced PrivilegedOperationExecutor and PrivilegedOperation(s) 
> which are meant to wrap invocations to the container-executor binary. 
> However, not all invocations of container-executor in LinuxContainerExecutor 
> use the PrivilegedOperationExecutor. We should change all such invocations to 
> use PrivilegedOperationExecutor.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to