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

Hudson commented on GIRAPH-900:
-------------------------------

ABORTED: Integrated in Giraph-trunk-Commit #1447 (See 
[https://builds.apache.org/job/Giraph-trunk-Commit/1447/])
GIRAPH-900: Remove timers for compute method (majakabiljo via pavanka) 
(pavanka: 
http://git-wip-us.apache.org/repos/asf?p=giraph.git&a=commit&h=e4b8db5d8b34b33d7551d912e53fe198bb95d7e0)
* giraph-core/src/main/java/org/apache/giraph/metrics/TimerDesc.java
* giraph-core/src/main/java/org/apache/giraph/graph/ComputeCallable.java
* giraph-core/src/main/java/org/apache/giraph/metrics/AggregatedMetrics.java
* 
giraph-core/src/main/java/org/apache/giraph/metrics/WorkerSuperstepMetrics.java
* giraph-core/src/main/java/org/apache/giraph/metrics/GiraphMetricsRegistry.java
* CHANGELOG


> Remove timers for compute method
> --------------------------------
>
>                 Key: GIRAPH-900
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-900
>             Project: Giraph
>          Issue Type: Improvement
>            Reporter: Maja Kabiljo
>            Assignee: Maja Kabiljo
>             Fix For: 1.1.0
>
>         Attachments: GIRAPH-900.patch
>
>
> When processing on a single vertex is quick, having timers around each 
> compute call is very expensive - in some experiments supersteps run up to 40% 
> faster without this timer. I never found it useful since average times are 
> usually short enough to not mean much, and also it's not just user code time 
> but when we block on sending requests it's counted here as well. So I'm just 
> removing it instead of making it optional.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to