Li Lu updated YARN-3210:
    Attachment: YARN-3210-030215_1.patch

Thanks [~zjshen]! I've updated my patch according to your review. Specifically, 
this time I refactored TimelineAggregatorsCollection for better aggregator 
creation interface. I've also gone through related classes for member variable 
names, including unit tests. 

> [Source organization] Refactor timeline aggregator according to new code 
> organization
> -------------------------------------------------------------------------------------
>                 Key: YARN-3210
>                 URL: https://issues.apache.org/jira/browse/YARN-3210
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>            Reporter: Li Lu
>            Assignee: Li Lu
>              Labels: refactor
>         Attachments: YARN-3210-022715.patch, YARN-3210-030215.patch, 
> YARN-3210-030215_1.patch
> We may want to refactor the code of timeline aggregator according to the 
> discussion of YARN-3166, the code organization for timeline service v2. We 
> need to refactor the code after we reach an agreement on the aggregator part 
> of YARN-3166. 

This message was sent by Atlassian JIRA

Reply via email to