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

ASF GitHub Bot commented on STORM-1571:
---------------------------------------

Github user darionyaphet commented on the pull request:

    https://github.com/apache/storm/pull/1143#issuecomment-187782060
  
    hi @unsleepy22 this metric is use to count time interval when fetch 
messages from Kafka Broker . In fetchMessage() , it's building a 
`FetchResponse` , consumer fetch messages from broker and pick out message set 
. I think it won't be return in 1ms :) 


> Improvment Kafka Spout Time Metric
> ----------------------------------
>
>                 Key: STORM-1571
>                 URL: https://issues.apache.org/jira/browse/STORM-1571
>             Project: Apache Storm
>          Issue Type: Improvement
>          Components: storm-kafka
>    Affects Versions: 0.9.3, 0.9.4, 0.9.5, 0.9.6
>         Environment: Mac 10.11.1 JDK 1.8.0_40
>            Reporter: darion yaphet
>            Assignee: darion yaphet
>            Priority: Minor
>         Attachments: 0001-Improvment-Kafka-Spout-Time-Metric.patch
>
>
> Use System.currentTimeMillis() to calculation time interval is better than 
> System.nanoTime() 



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

Reply via email to