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

Zhijie Shen commented on YARN-3051:
-----------------------------------

bq. One more thing I have changed is limit of entities by default have been 
kept has 100 instead of 1000. 1000 seemed too many. Thoughts ?

Sure, just noticed that the previous limit is 100 too.

bq. Ok, so the last entity entry should be the one returned ?

It's not that straightforward. For example, I can put entity 1 twice: one 
contains event 1 and the other contains event 2. In fact, when I want to 
retrieve the entity 1 with event field included. I actually want to have both 
events. I can see two choices: one is to merge the entity data at the write 
path and the other at the read path.

bq. Added in TimelineUtils because dumpTimelineRecordtoJSON used by FS Writer 
was also put in the same class.

That method is used by downstream project (e.g., tez) to logging/debugging the 
ATS integration. And this all getters of the data model objects are annotated. 
The method is applicable to all these objects. On the other side, we only 
annotate "jasonsetter" for TimelineEntity, such that getTimelineRecordFromJSON 
is not generalized enough for all purpose, but for FS impl only now. Maybe we 
can hold back the method and promote it to public api once we see real use case 
of it. Thoughts?

> [Storage abstraction] Create backing storage read interface for ATS readers
> ---------------------------------------------------------------------------
>
>                 Key: YARN-3051
>                 URL: https://issues.apache.org/jira/browse/YARN-3051
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>    Affects Versions: YARN-2928
>            Reporter: Sangjin Lee
>            Assignee: Varun Saxena
>         Attachments: YARN-3051-YARN-2928.003.patch, 
> YARN-3051-YARN-2928.03.patch, YARN-3051-YARN-2928.04.patch, 
> YARN-3051-YARN-2928.05.patch, YARN-3051.Reader_API.patch, 
> YARN-3051.Reader_API_1.patch, YARN-3051.Reader_API_2.patch, 
> YARN-3051.Reader_API_3.patch, YARN-3051.Reader_API_4.patch, 
> YARN-3051.wip.02.YARN-2928.patch, YARN-3051.wip.patch, YARN-3051_temp.patch
>
>
> Per design in YARN-2928, create backing storage read interface that can be 
> implemented by multiple backing storage implementations.



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

Reply via email to