giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Avery Ching" <avery.ch...@gmail.com>
Subject Re: Review Request: YourKit profiling API for easy profiling of giraph
Date Mon, 08 Apr 2013 17:27:51 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10147/#review18781
-----------------------------------------------------------


Overall, I think it's great that we make toolkits like Yourkit easy to use in Giraph and this
is a good first step.  Personally I feel a little strange though with adding the actual logging
calls in the usercode.  Also, I am a bit hesitant on the maven dependencies that we will have
to maintain going forward...What do others think here?


giraph-core/pom.xml
<https://reviews.apache.org/r/10147/#comment39201>

    Same here?



pom.xml
<https://reviews.apache.org/r/10147/#comment39200>

    Are we maintaining this package?


- Avery Ching


On March 27, 2013, 7:14 a.m., Nitay Joffe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10147/
> -----------------------------------------------------------
> 
> (Updated March 27, 2013, 7:14 a.m.)
> 
> 
> Review request for giraph.
> 
> 
> Description
> -------
> 
> https://issues.apache.org/jira/browse/GIRAPH-592
> 
> 
> Diffs
> -----
> 
>   giraph-core/pom.xml 3580d0cd3cd943e0762b3eb069183a31fc690300 
>   giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java ffcae6e79cf0bd96745ef9bc9fc420f5b28494b2

>   giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java 7882d06fddd70eb9fef7a8d0463c6618b39d3d4d

>   giraph-core/src/main/java/org/apache/giraph/utils/YourKitContext.java PRE-CREATION

>   giraph-core/src/main/java/org/apache/giraph/utils/YourKitProfiler.java PRE-CREATION

>   giraph-core/src/main/java/org/apache/giraph/worker/BspServiceWorker.java 74c1f874eea819b5e4fafd684a88f17eaded7dc5

>   pom.xml e576e4b5e55be507225860e7d1c1fc55629f1761 
> 
> Diff: https://reviews.apache.org/r/10147/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nitay Joffe
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message