You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by revans2 <gi...@git.apache.org> on 2017/10/17 15:04:37 UTC

[GitHub] storm issue #2377: [STORM-2780] MetricsConsumer record unnecessary timestamp

Github user revans2 commented on the issue:

    https://github.com/apache/storm/pull/2377
  
    @BigOneLiu When the task info is created
    
    https://github.com/apache/storm/blob/124acb92dff04a57b530ab4d95a698abc8ff46d9/storm-client/src/jvm/org/apache/storm/executor/Executor.java#L291-L293
    
    The time that is set there is when the metric was sent (in seconds).  In most cases it should not take much if any time for it to make its way to the metrics consumer, but if it does the times could be off from one another.
    
    I personally am +1 for this change, because I don't really use this time when I am looking at logged metrics, but if others want it they should speak up.


---