You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@slider.apache.org by sm...@apache.org on 2015/03/16 17:48:46 UTC

[13/19] incubator-slider git commit: SLIDER-319 -block out ganglia integration

SLIDER-319 -block out ganglia integration


Project: http://git-wip-us.apache.org/repos/asf/incubator-slider/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-slider/commit/148a4354
Tree: http://git-wip-us.apache.org/repos/asf/incubator-slider/tree/148a4354
Diff: http://git-wip-us.apache.org/repos/asf/incubator-slider/diff/148a4354

Branch: refs/heads/feature/package_simplification_II
Commit: 148a4354d7fe04b71c527fc8d81968741b27ee65
Parents: e6c988f
Author: Steve Loughran <st...@apache.org>
Authored: Thu Mar 12 15:54:53 2015 -0700
Committer: Steve Loughran <st...@apache.org>
Committed: Thu Mar 12 15:54:53 2015 -0700

----------------------------------------------------------------------
 slider-core/pom.xml                               |  2 ++
 .../management/MetricsBindingService.java         | 18 ++++++++++--------
 2 files changed, 12 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/148a4354/slider-core/pom.xml
----------------------------------------------------------------------
diff --git a/slider-core/pom.xml b/slider-core/pom.xml
index 0633446..22bc868 100644
--- a/slider-core/pom.xml
+++ b/slider-core/pom.xml
@@ -343,10 +343,12 @@
       <artifactId>metrics-core</artifactId>
     </dependency>
 
+<!--
     <dependency>
       <groupId>com.codahale.metrics</groupId>
       <artifactId>metrics-ganglia</artifactId>
     </dependency>
+-->
 
     <dependency>
       <groupId>com.codahale.metrics</groupId>

http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/148a4354/slider-core/src/main/java/org/apache/slider/server/appmaster/management/MetricsBindingService.java
----------------------------------------------------------------------
diff --git a/slider-core/src/main/java/org/apache/slider/server/appmaster/management/MetricsBindingService.java b/slider-core/src/main/java/org/apache/slider/server/appmaster/management/MetricsBindingService.java
index fb1dbd2..f8646bf 100644
--- a/slider-core/src/main/java/org/apache/slider/server/appmaster/management/MetricsBindingService.java
+++ b/slider-core/src/main/java/org/apache/slider/server/appmaster/management/MetricsBindingService.java
@@ -22,9 +22,7 @@ import com.codahale.metrics.JmxReporter;
 import com.codahale.metrics.MetricRegistry;
 import com.codahale.metrics.ScheduledReporter;
 import com.codahale.metrics.Slf4jReporter;
-import com.codahale.metrics.ganglia.GangliaReporter;
 import com.google.common.base.Preconditions;
-import info.ganglia.gmetric4j.gmetric.GMetric;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.service.CompositeService;
 import org.apache.slider.server.services.workflow.ClosingService;
@@ -86,23 +84,26 @@ public class MetricsBindingService extends CompositeService
 
     // Ganglia
     if (conf.getBoolean(METRICS_GANGLIA_ENABLED, false)) {
-      GangliaReporter gangliaReporter;
+      log.warn("Ganglia integration is not implemented");
+/*
+      // This is all disabled due to transitive dependencies on an LGPL library
+      com.codahale.metrics.ganglia.GangliaReporter gangliaReporter;
       String host = conf.getTrimmed(METRICS_GANGLIA_HOST, "");
       int port = conf.getInt(METRICS_GANGLIA_PORT, DEFAULT_GANGLIA_PORT);
       int interval = conf.getInt(METRICS_GANGLIA_REPORT_INTERVAL, 60);
       int ttl = 1;
-      GMetric.UDPAddressingMode
-          mcast = GMetric.UDPAddressingMode.getModeForAddress(host);
+      info.ganglia.gmetric4j.gmetric.GMetric.UDPAddressingMode
+          mcast = info.ganglia.gmetric4j.gmetric.GMetric.UDPAddressingMode.getModeForAddress(host);
       boolean ganglia31 = conf.getBoolean(METRICS_GANGLIA_VERSION_31, true);
 
-      final GMetric ganglia =
-          new GMetric(
+      final info.ganglia.gmetric4j.gmetric.GMetric ganglia =
+          new info.ganglia.gmetric4j.gmetric.GMetric(
               host,
               port,
               mcast,
               ttl,
               ganglia31);
-      gangliaReporter = GangliaReporter.forRegistry(metrics)
+      gangliaReporter = com.codahale.metrics.ganglia.GangliaReporter.forRegistry(metrics)
                                        .convertRatesTo(TimeUnit.SECONDS)
                                        .convertDurationsTo(
                                            TimeUnit.MILLISECONDS)
@@ -111,6 +112,7 @@ public class MetricsBindingService extends CompositeService
       addService(new ClosingService<ScheduledReporter>(gangliaReporter));
       summary.append(String.format(", Ganglia at %s:%d interval=%d",
           host, port, interval));
+      */
     }
 
     // Logging