[jira] [Updated] (YARN-710) Add to ser/deser methods to RecordFactory

["Alejandro Abdelnur (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/YARN-710?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alejandro Abdelnur updated YARN-710:
------------------------------------

    Attachment: YARN-710.patch

Adding write()/read() methods to the RecordFactory.

The implementation rides on naming conventions and reflection as there is no API contract for such operations.

Note that we cannot use ProtoBase to get the Message because not all record PBImpls extend ProtoBase. Also, even if they do, the contructor of the PBImpl getting he PROTO still would have to invoked via reflection.

IMO, the ideal solution is what Bobby suggested, exposing PROTO classes directly, but as he indicates it is a difficult undertaking. Thus, for now, sticking to the minimum to expose this functionality.
                
> Add to ser/deser methods to RecordFactory
> -----------------------------------------
>
>                 Key: YARN-710
>                 URL: https://issues.apache.org/jira/browse/YARN-710
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: api
>    Affects Versions: 2.0.4-alpha
>            Reporter: Alejandro Abdelnur
>            Assignee: Alejandro Abdelnur
>             Fix For: 2.0.5-beta
>
>         Attachments: YARN-710.patch
>
>
> I order to do things like AMs failover and checkpointing I need to serialize app IDs, app attempt IDs, containers and/or IDs,  resource requests, etc.
> Because we are wrapping/hiding the PB implementation from the APIs, we are hiding the built in PB ser/deser capabilities.

--
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