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

Yan Fang commented on SAMZA-444:
--------------------------------

It overall looks good for me. With the models, the code looks much clear for 
me. Since it's a big change. You may also want to hear others feedback. 

> Provide a Samza job data model for job coordinator
> --------------------------------------------------
>
>                 Key: SAMZA-444
>                 URL: https://issues.apache.org/jira/browse/SAMZA-444
>             Project: Samza
>          Issue Type: Sub-task
>          Components: container, yarn
>    Affects Versions: 0.9.0
>            Reporter: Chris Riccomini
>            Assignee: Chris Riccomini
>             Fix For: 0.9.0
>
>         Attachments: SAMZA-444-0.patch
>
>
> We currently have no data model for Samza jobs. As a result, we have really 
> hacky code in JsonHelpers to handle (de)serializing job/container/task 
> information between the job coordinator and its SamzaContainers.
> We should provide a proper data model for Samza jobs, which includes a Job, 
> Container, and Task. The objects should contain relevant data that the job 
> coordinator and SamzaContainer need. These classes should be written in Java, 
> and use proper Jackson annotations to allow for easy (de)serialization.



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

Reply via email to