Author: mattsicker
Date: Tue Apr 15 05:15:47 2014
New Revision: 1587419
URL: http://svn.apache.org/r1587419
Log:
Add logger and more terse reflection.
Modified:
logging/log4j/log4j2/trunk/log4j-core/src/test/java/org/apache/logging/log4j/core/util/Profiler.java
Modified:
logging/log4j/log4j2/trunk/log4j-core/src/test/java/org/apache/logging/log4j/core/util/Profiler.java
URL:
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/test/java/org/apache/logging/log4j/core/util/Profiler.java?rev=1587419&r1=1587418&r2=1587419&view=diff
==============================================================================
---
logging/log4j/log4j2/trunk/log4j-core/src/test/java/org/apache/logging/log4j/core/util/Profiler.java
(original)
+++
logging/log4j/log4j2/trunk/log4j-core/src/test/java/org/apache/logging/log4j/core/util/Profiler.java
Tue Apr 15 05:15:47 2014
@@ -16,28 +16,31 @@
*/
package org.apache.logging.log4j.core.util;
-import java.lang.reflect.Field;
+import org.apache.logging.log4j.Logger;
+import org.apache.logging.log4j.core.helpers.Loader;
+import org.apache.logging.log4j.status.StatusLogger;
/**
* YourKit Java Profiler helper class.
*/
public final class Profiler {
+ private static final Logger LOGGER = StatusLogger.getLogger();
+
private static Object profiler;
private static Class<?> profilingModes;
private static Class<?> controllerClazz;
static {
try {
- controllerClazz = Class.forName("com.yourkit.api.Controller");
- profilingModes = Class.forName("com.yourkit.api.ProfilingModes");
+ controllerClazz = Loader.loadClass("com.yourkit.api.Controller");
+ profilingModes =
Loader.loadClass("com.yourkit.api.ProfilingModes");
try {
- profiler = controllerClazz.newInstance();
+ profiler = controllerClazz.getConstructor().newInstance();
} catch (final Exception e) {
- e.printStackTrace();
- System.out.println("Profiler was active, but failed due: " +
e.getMessage());
+ LOGGER.error("Profiler was active, but failed.", e);
}
}
- catch (final Exception e) {
+ catch (final Exception ignored) {
// Ignore
}
@@ -50,20 +53,24 @@ public final class Profiler {
return profiler != null;
}
+ private static long cpuSampling() throws NoSuchFieldException,
IllegalAccessException {
+ return
profilingModes.getDeclaredField("CPU_SAMPLING").getLong(profilingModes);
+ }
+
+ private static long snapshotWithoutHeap() throws NoSuchFieldException,
IllegalAccessException {
+ return
profilingModes.getDeclaredField("SNAPSHOT_WITHOUT_HEAP").getLong(profilingModes);
+ }
+
public static void start() {
if (profiler != null) {
try {
- final Field f =
profilingModes.getDeclaredField("CPU_SAMPLING");
- final Object[] args = new Object[2];
- args[0] = f.getLong(profilingModes);
- args[1] = "";
- final Class<?>[] parms = new Class<?>[] {long.class,
String.class};
- controllerClazz.getMethod("startCPUProfiling",
parms).invoke(profiler, args);
+ controllerClazz
+ .getMethod("startCPUProfiling", long.class,
String.class)
+ .invoke(profiler, cpuSampling(), "");
}
catch (final Exception e) {
- e.printStackTrace();
- System.out.println("Profiler was active, but failed due: " +
e.getMessage());
+ LOGGER.error("Profiler was active, but failed.", e);
}
}
}
@@ -71,16 +78,15 @@ public final class Profiler {
public static void stop() {
if (profiler != null) {
try {
- final Field f =
profilingModes.getDeclaredField("SNAPSHOT_WITHOUT_HEAP");
- final Object[] args = new Object[1];
- args[0] = f.getLong(profilingModes);
- final Class<?>[] parms = new Class<?>[] {long.class};
- profiler.getClass().getMethod("captureSnapshot",
parms).invoke(profiler, args);
-
profiler.getClass().getMethod("stopCPUProfiling").invoke(profiler);
+ controllerClazz
+ .getMethod("captureSnapshot", long.class)
+ .invoke(profiler, snapshotWithoutHeap());
+ controllerClazz
+ .getMethod("stopCPUProfiling")
+ .invoke(profiler);
}
catch (final Exception e) {
- e.printStackTrace();
- System.out.println("Profiler was active, but failed due: " +
e.getMessage());
+ LOGGER.error("Profiler was active, but failed.", e);
}
}
}
