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

Sangjin Lee edited comment on YARN-5169 at 5/27/16 8:11 PM:
------------------------------------------------------------

Thanks for your comments [~gtCarrera9]! I agree that using 
{{SystemMetricEvent}} and others to populate the timestamp is one approach to 
this. I have a bit broader question for the event timestamps, however. Are 
event timestamps for consumption by the timeline service exclusively? Or should 
we expect them to have reasonable timestamps more generally?

And also is the rationale of not generating the timestamp by default (being too 
expensive) still a valid conclusion?


was (Author: sjlee0):
Thanks for your comments [~gtCarrera9]! I agree that using 
{{SystemMetricEvent}} and others to populate the timestamp is one approach to 
this. I have a bit broader question for the event timestamps, however. Are 
event timestamps for consumption by the timeline service exclusively? Or should 
we expect them to have reasonable timestamps more generally?

> most YARN events have timestamp of -1
> -------------------------------------
>
>                 Key: YARN-5169
>                 URL: https://issues.apache.org/jira/browse/YARN-5169
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: yarn
>    Affects Versions: 2.7.2
>            Reporter: Sangjin Lee
>
> Most of the YARN events (subclasses of {{AbstractEvent}}) have timestamp of 
> -1. {{AbstractEvent}} have two constructors, one that initializes the 
> timestamp to -1 and the other to the caller-provided value. But most events 
> use the former (thus timestamp of -1).
> Some of the more common events, including {{ApplicationEvent}}, 
> {{ContainerEvent}}, {{JobEvent}}, etc. do not set the timestamp.
> The rationale for this behavior seems to be mentioned in {{AbstractEvent}}:
> {code}
>   // use this if you DON'T care about the timestamp
>   public AbstractEvent(TYPE type) {
>     this.type = type;
>     // We're not generating a real timestamp here.  It's too expensive.
>     timestamp = -1L;
>   }
> {code}
> This absence of the timestamp isn't really visible in many cases and 
> therefore may have gone unnoticed, but the timeline service exposes this 
> problem very visibly.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to