Robert Kanter commented on YARN-3264:

Looks good.

Two minor thing:
- {{TIMELINE_SERVICE_STORAGAGE_EXTENSION}} in {{FileSystemTimelineWriterImpl}} 
has a typo.
- Would it make sense to rename {{TimelineAggregationGranularity}} to 
{{TimelineAggregationTrack}}?  That seems more clear about what it's for, and 
more in line with what all the comments say about it.

> [Storage implementation] Create backing storage write interface and  a POC 
> only file based storage implementation
> -----------------------------------------------------------------------------------------------------------------
>                 Key: YARN-3264
>                 URL: https://issues.apache.org/jira/browse/YARN-3264
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>            Reporter: Vrushali C
>            Assignee: Vrushali C
>         Attachments: YARN-3264.001.patch, YARN-3264.002.patch, 
> YARN-3264.003.patch, YARN-3264.004.patch, YARN-3264.005.patch, 
> YARN-3264.006.patch, YARN-3264.007.patch
> For the PoC, need to create a backend impl for file based storage of entities 

This message was sent by Atlassian JIRA

Reply via email to