[ 
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

Addressing Sid's first 2 bullet items. On the 3rd one, that would require 
assuming that I could remove 'PBImpl' from the class, add 'Proto' and then I 
get the proto implementation. Plus package swithing. While the 'PBImpl' postfix 
is used as a convention already, the 'Proto' and the package are not used as 
convention, thus I'd leave it as it is.

Reading writing multiple records is responsibility of the user.

On how to mark records that are serializable? My take is that either we add 
static read/write methods in all the ones we want that. Or we have a 'SerDeser' 
interface and a SerDserUtils with the read/write. The chosen impl would 
delegate to the factory.
                
> 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
>         Attachments: YARN-710.patch, 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

Reply via email to