[
https://issues.apache.org/jira/browse/YARN-5127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15302884#comment-15302884
]
Arun Suresh commented on YARN-5127:
-----------------------------------
Thanks for the quick update [~hrsharma]..
I guess the remain checkstyle warning is also fixable.. as well as the first
javadoc error.
It would be good to update the
{{TestDistributedSchedulingService::testRPCWrapping}} to ensure the
ExecutionType is actually sent over the wire..
> Expose ExecutionType in Container api record
> --------------------------------------------
>
> Key: YARN-5127
> URL: https://issues.apache.org/jira/browse/YARN-5127
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Arun Suresh
> Assignee: Hitesh Sharma
> Attachments: YARN-5127.002.patch, YARN-5127.003.patch,
> YARN-5127.v1.patch
>
>
> Currently the ExecutionType of the Container returned as a response to the
> allocate call is contained in the {{ContinerTokenIdentifier}} which is
> encoded into the ContainerToken.
> Unfortunately, the client would need to decode the returned token to access
> the ContainerTokenIdentifier, which probably should not be allowed.
> This JIRA proposes to add a {{getExecutionType()}} method in the container
> record.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
