Fixed circular dependency problem which caused an ExceptionInInitiali…

…zer error
tomdz · Aug 11, 2011 · bf55150 · bf55150
1 parent ee0d350
commit bf55150
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/metrics-core/src/main/java/com/yammer/metrics/Metrics.java b/metrics-core/src/main/java/com/yammer/metrics/Metrics.java
@@ -16,7 +16,7 @@
 public class Metrics {
     private static final MetricsRegistry DEFAULT_REGISTRY = new MetricsRegistry();
     static {{
-        JmxReporter.INSTANCE.start();
+        JmxReporter.startDefault(DEFAULT_REGISTRY);
         // make sure we initialize this so it can monitor GC etc
         VirtualMachineMetrics.daemonThreadCount();
     }}

diff --git a/metrics-core/src/main/java/com/yammer/metrics/reporting/JmxReporter.java b/metrics-core/src/main/java/com/yammer/metrics/reporting/JmxReporter.java
@@ -291,10 +291,10 @@ public List<?> values() {
         }
     }
 
-    public static final JmxReporter INSTANCE = new JmxReporter();
-
-    /*package*/ JmxReporter() {
-        this(Metrics.defaultRegistry());
+    private static JmxReporter INSTANCE;
+    public static final void startDefault(MetricsRegistry defaultMetricsRegistry) {
+        INSTANCE = new JmxReporter(defaultMetricsRegistry);
+        INSTANCE.start();
     }
 
     public JmxReporter(MetricsRegistry metricsRegistry) {