[jboss-cvs] jboss-profiler/java/src/expansion/org/jboss/profiler/exp/agent ...

Takuro Okada t2-okada at nri.co.jp
Thu Feb 15 03:29:02 EST 2007


  User: tokada  
  Date: 07/02/15 03:29:02

  Modified:    java/src/expansion/org/jboss/profiler/exp/agent  Tag:
                        JBossProfiler_Expansion DefaultProfilerService.java
  Log:
  Corrected some bugs.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.6   +7 -5      jboss-profiler/java/src/expansion/org/jboss/profiler/exp/agent/Attic/DefaultProfilerService.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: DefaultProfilerService.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-profiler/java/src/expansion/org/jboss/profiler/exp/agent/Attic/DefaultProfilerService.java,v
  retrieving revision 1.1.2.5
  retrieving revision 1.1.2.6
  diff -u -b -r1.1.2.5 -r1.1.2.6
  --- DefaultProfilerService.java	28 Nov 2006 02:47:49 -0000	1.1.2.5
  +++ DefaultProfilerService.java	15 Feb 2007 08:29:02 -0000	1.1.2.6
  @@ -113,7 +113,9 @@
           collector.setCollectorConfig(collectorConfig);
           if(collectorExecutor == null) return;
           try {
  +            synchronized(metricsMap) {
               collectorExecutor.execute(collector);
  +            }
           } catch (Exception e) {
               logger.error("failed to run collector["+collector.getClass().getName()+"].");
           }
  @@ -164,14 +166,14 @@
       @SuppressWarnings("unchecked")
       private MetricsMap copyMetrics() {
           MetricsMap result = null;
  -        synchronized(metricsMap) {
               try {
  +            synchronized(metricsMap) {
                   // TODO use Metric#clone()
                   result = (MetricsMap)CopyUtil.deepCopy(metricsMap);
  +            }
               } catch (Exception e) {
                   logger.error("failed to copy the metrics data.");
               }
  -        }
           return result;
       }
   
  
  
  



More information about the jboss-cvs-commits mailing list