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

Avery Ching commented on GIRAPH-171:
------------------------------------

+1 Argh, it is inconsistent with the counter, GIRAPH_TIMERS_COUNTER_GROUP_NAME. 
 Thanks for the fix Eugene!
                
> total time in MasterThread.run() is calculated incorrectly
> ----------------------------------------------------------
>
>                 Key: GIRAPH-171
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-171
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Eugene Koontz
>            Assignee: Eugene Koontz
>         Attachments: GIRAPH-171.patch
>
>
> While running PageMarkBenchMark, I was seeing in the output:
> {{graph.MasterThread(172): total: Took 1.3336739262910001E9 seconds.}}
> This was because currently, in {{MasterThread.run()}}, we have:
> {code}
> LOG.info("total: Took " +
>  ((System.currentTimeMillis() / 1000.0d) -
>  setupSecs) + " seconds.");
> {code}
> but it should be:
> {code}
>            LOG.info("total: Took " +
>                ((System.currentTimeMillis() - startMillis) /
>               1000.0d) + " seconds.");
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to