Author: nicolas Date: Thu Feb 19 13:19:41 2009 New Revision: 745851 URL: http://svn.apache.org/viewvc?rev=745851&view=rev Log: fix retreive metrics by name / role
Modified: commons/sandbox/monitoring/branches/modules/core/src/main/java/org/apache/commons/monitoring/monitors/AbstractMonitor.java Modified: commons/sandbox/monitoring/branches/modules/core/src/main/java/org/apache/commons/monitoring/monitors/AbstractMonitor.java URL: http://svn.apache.org/viewvc/commons/sandbox/monitoring/branches/modules/core/src/main/java/org/apache/commons/monitoring/monitors/AbstractMonitor.java?rev=745851&r1=745850&r2=745851&view=diff ============================================================================== --- commons/sandbox/monitoring/branches/modules/core/src/main/java/org/apache/commons/monitoring/monitors/AbstractMonitor.java (original) +++ commons/sandbox/monitoring/branches/modules/core/src/main/java/org/apache/commons/monitoring/monitors/AbstractMonitor.java Thu Feb 19 13:19:41 2009 @@ -36,7 +36,6 @@ public abstract class AbstractMonitor implements Monitor { - @SuppressWarnings("unchecked") private final ConcurrentMap<Role, Metric> metrics; private final Key key; @@ -53,7 +52,6 @@ * * @return the ConcurrentMap implementation to use for storing metrics */ - @SuppressWarnings("unchecked") protected ConcurrentHashMap<Role, Metric> createConcurrentMap() { return new ConcurrentHashMap<Role, Metric>(); @@ -72,7 +70,7 @@ */ public final Metric getMetric( String role ) { - return metrics.get( role ); + return metrics.get( Role.getRole( role ) ); } public final Metric getMetric( Role role ) @@ -80,7 +78,6 @@ return metrics.get( role ); } - @SuppressWarnings("unchecked") public final Collection<Role> getRoles() { return Collections.unmodifiableCollection( metrics.keySet() ); @@ -115,13 +112,11 @@ } } - @SuppressWarnings("unchecked") public Counter getCounter( String role ) { return getCounter( Role.getRole( role ) ); } - @SuppressWarnings("unchecked") public Gauge getGauge( String role ) { return getGauge( Role.getRole( role ) );