Kill unused classes GaugeExporter, CounterMetricJMXExporter, GaugeMXB…

…ean, MutableGauge (#491)
salesforce · Jun 21, 2019 · f44aefd · f44aefd
1 parent e1cf4b1
commit f44aefd
Show file tree

Hide file tree

Showing 6 changed files with 1 addition and 307 deletions.
diff --git a/ArgusCore/src/main/java/com/salesforce/dva/argus/entity/GaugeMXBean.java b/ArgusCore/src/main/java/com/salesforce/dva/argus/entity/GaugeMXBean.java
diff --git a/ArgusCore/src/main/java/com/salesforce/dva/argus/entity/MutableGauge.java b/ArgusCore/src/main/java/com/salesforce/dva/argus/entity/MutableGauge.java
diff --git a/...Core/src/main/java/com/salesforce/dva/argus/service/monitor/CounterMetricJMXExporter.java b/...Core/src/main/java/com/salesforce/dva/argus/service/monitor/CounterMetricJMXExporter.java
diff --git a/ArgusCore/src/main/java/com/salesforce/dva/argus/service/monitor/DefaultMonitorService.java b/ArgusCore/src/main/java/com/salesforce/dva/argus/service/monitor/DefaultMonitorService.java
@@ -117,7 +117,6 @@ public class DefaultMonitorService extends DefaultJPAService implements MonitorS
 	private final DashboardService _dashboardService;
 	private final MetricService _metricService;
 	private final MailService _mailService;
-	private final GaugeExporter _gaugeExporter;
 	private final Map<Metric, GaugeMetric> _metrics;
 	private final Map<Metric, MetricMXBean> _registeredMetrics;
 	private final PrincipalUser _adminUser;
@@ -144,7 +143,7 @@ public class DefaultMonitorService extends DefaultJPAService implements MonitorS
 	@Inject
 	public DefaultMonitorService(TSDBService tsdbService, UserService userService, AlertService alertService,
 			ServiceManagementService serviceManagementService, DashboardService dashboardService, MetricService metricService, MailService mailService, 
-			GaugeExporter gaugeExporter, SystemConfiguration sysConfig) {
+			SystemConfiguration sysConfig) {
 		super(null, sysConfig);
 		requireArgument(tsdbService != null, "TSDB service cannot be null.");
 		requireArgument(userService != null, "User service cannot be null.");
@@ -160,7 +159,6 @@ public DefaultMonitorService(TSDBService tsdbService, UserService userService, A
 		_metricService = metricService;
 		_mailService = mailService;
 		_adminUser = _userService.findAdminUser();
-		_gaugeExporter = gaugeExporter;
 		_mbeanServer = ManagementFactory.getPlatformMBeanServer();
 		_metrics = new ConcurrentHashMap<>();
 		_registeredMetrics = new ConcurrentHashMap<>();

diff --git a/ArgusCore/src/main/java/com/salesforce/dva/argus/service/monitor/GaugeExporter.java b/ArgusCore/src/main/java/com/salesforce/dva/argus/service/monitor/GaugeExporter.java
diff --git a/ArgusCore/src/main/java/com/salesforce/dva/argus/system/SystemInitializer.java b/ArgusCore/src/main/java/com/salesforce/dva/argus/system/SystemInitializer.java
@@ -54,9 +54,7 @@
 import com.salesforce.dva.argus.service.jpa.DefaultServiceManagementService;
 import com.salesforce.dva.argus.service.management.DefaultManagementService;
 import com.salesforce.dva.argus.service.metric.AsyncMetricService;
-import com.salesforce.dva.argus.service.monitor.CounterMetricJMXExporter;
 import com.salesforce.dva.argus.service.monitor.DefaultMonitorService;
-import com.salesforce.dva.argus.service.monitor.GaugeExporter;
 import com.salesforce.dva.argus.service.oauth.DefaultOAuthAuthorizationCodeService;
 import com.salesforce.dva.argus.service.schema.DefaultDiscoveryService;
 import com.salesforce.dva.argus.service.tsdb.CachedTSDBService;
@@ -229,7 +227,6 @@ private void configureLogging() {
     }
 
     private void configureServices() {
-        bindConcreteClass(CounterMetricJMXExporter.class, GaugeExporter.class);
         bindConcreteClass(Property.CACHE_SERVICE_IMPL_CLASS, CacheService.class);
         bindConcreteClass(Property.MQ_SERVICE_IMPL_CLASS, MQService.class);
         bindConcreteClass(Property.ALERT_SERVICE_IMPL_CLASS, AlertService.class);