zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eolivelli <...@git.apache.org>
Subject [GitHub] zookeeper pull request #601: WIP - ZOOKEEPER-3123 MetricsProvider Lifecycle ...
Date Mon, 27 Aug 2018 14:47:50 GMT
Github user eolivelli commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/601#discussion_r213002472
  
    --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServerMain.java ---
    @@ -117,13 +121,23 @@ public void runFromConfig(ServerConfig config)
             LOG.info("Starting server");
             FileTxnSnapLog txnLog = null;
             try {
    +            try {
    +                metricsProvider = MetricsProviderBootstrap
    +                        .startMetricsProvider(config.metricsProviderClassName, new Properties());
    +            } catch (MetricsProviderLifeCycleException error) {
    +                LOG.error("Cannot boot MetricsProvider {}", config.metricsProviderClassName,
error);
    +                throw new IOException("Cannot boot MetricsProvider "+config.metricsProviderClassName,
    --- End diff --
    
    @lvfangmin I can change it if you feel strong.


---

Mime
View raw message