fix error when no dep dropwizard java.lang.NoClassDefFoundError: com/codahale/metrics/Reservoir

Author: songxiaosheng

zookeeper-server/src/main/java/org/apache/zookeeper/server/ZooKeeperServerMain.java

@@ -22,6 +22,7 @@
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 import javax.management.JMException;
+
 import org.apache.yetus.audience.InterfaceAudience;
 import org.apache.zookeeper.audit.ZKAuditProvider;
 import org.apache.zookeeper.jmx.ManagedUtil;
@@ -115,6 +116,7 @@ protected void initializeAndRun(String[] args) throws ConfigException, IOExcepti
 
     /**
      * Run from a ServerConfig.
+     *
      * @param config ServerConfig to use.
      * @throws IOException
      * @throws AdminServerException
@@ -125,12 +127,13 @@ public void runFromConfig(ServerConfig config) throws IOException, AdminServerEx
         try {
             try {
                 metricsProvider = MetricsProviderBootstrap.startMetricsProvider(
-                    config.getMetricsProviderClassName(),
-                    config.getMetricsProviderConfiguration());
+                        config.getMetricsProviderClassName(),
+                        config.getMetricsProviderConfiguration());
             } catch (MetricsProviderLifeCycleException error) {
                 throw new IOException("Cannot boot MetricsProvider " + config.getMetricsProviderClassName(), error);
             }
-            ServerMetrics.metricsProviderInitialized(metricsProvider);
+
+            metricsProviderInitialized();
             ProviderRegistry.initialize();
             // Note that this thread isn't going to be doing anything else,
             // so rather than spawning another thread, we will just call
@@ -169,11 +172,11 @@ public void runFromConfig(ServerConfig config) throws IOException, AdminServerEx
             }
 
             containerManager = new ContainerManager(
-                zkServer.getZKDatabase(),
-                zkServer.firstProcessor,
-                Integer.getInteger("znode.container.checkIntervalMs", (int) TimeUnit.MINUTES.toMillis(1)),
-                Integer.getInteger("znode.container.maxPerMinute", 10000),
-                Long.getLong("znode.container.maxNeverUsedIntervalMs", 0)
+                    zkServer.getZKDatabase(),
+                    zkServer.firstProcessor,
+                    Integer.getInteger("znode.container.checkIntervalMs", (int) TimeUnit.MINUTES.toMillis(1)),
+                    Integer.getInteger("znode.container.maxPerMinute", 10000),
+                    Long.getLong("znode.container.maxNeverUsedIntervalMs", 0)
             );
             containerManager.start();
             ZKAuditProvider.addZKStartStopAuditLog();
@@ -210,6 +213,18 @@ public void runFromConfig(ServerConfig config) throws IOException, AdminServerEx
         }
     }
 
+    private void metricsProviderInitialized() {
+        try {
+            ServerMetrics.metricsProviderInitialized(metricsProvider);
+        } catch (Exception e) {
+            if (e instanceof ClassNotFoundException) {
+                LOG.warn("Metrics provider not found, metrics will not be reported");
+                return;
+            }
+            throw e;
+        }
+    }
+
     /**
      * Shutdown the serving instance
      */