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

Vinod Kumar Vavilapalli commented on YARN-708:
----------------------------------------------

The patch looked daunting at first, but now after deeper looks, seems 
manageable.

One thing that caught my attention is the configuration changes - the generic 
serialization config is removed, and we still have to set three configuration 
items. With little more work, we can have a single config item drive everything 
- essentially record serialization, server and client are all tied together. 
But perhaps in future.

+1, will check it in if and when Jenkins is okay too.
                
> Move RecordFactory classes to hadoop-yarn-api, miscellaneous fixes to the 
> interfaces
> ------------------------------------------------------------------------------------
>
>                 Key: YARN-708
>                 URL: https://issues.apache.org/jira/browse/YARN-708
>             Project: Hadoop YARN
>          Issue Type: Task
>    Affects Versions: 2.0.4-alpha
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: move_YARN_708.1.sh, YARN-708.1.txt, YARN-708.full.1.txt, 
> YARN-708_postMvScript.1.txt
>
>
> This is required for additional changes in YARN-528. 
> Some of the interfaces could use some cleanup as well - they shouldn't be 
> declaring YarnException (Runtime) in their signature.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to