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