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 ) );