[
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)