Fix failures

coherence-visualvm-plugin/pom.xml

@@ -130,14 +130,6 @@
       <artifactId>coherence-discovery</artifactId>
     </dependency>
 
-    <!--- mock dependency -->
-    <dependency>
-      <groupId>com.oracle.coherence.plugin.visualvm</groupId>
-      <artifactId>coherence-visualvm-tracer</artifactId>
-      <version>1.7.0-SNAPSHOT</version>
-      <scope>provided</scope>
-    </dependency>
-
     <dependency>
       <groupId>org.hamcrest</groupId>
       <artifactId>hamcrest</artifactId>

coherence-visualvm-plugin/src/main/java/com/oracle/coherence/plugin/visualvm/VisualVMInstaller.java

@@ -65,12 +65,12 @@ public void restored()
         CoherenceApplicationTypeFactory.initialize();
 
         // register the tracer probes
-        if (provider == null)
+        if (m_provider == null)
             {
-            provider = new TracerPackageProviderImpl();
+            m_provider = new TracerPackageProviderImpl();
             }
 
-        TracerSupport.getInstance().registerPackageProvider(provider);
+        TracerSupport.getInstance().registerPackageProvider(m_provider);
         }
 
     /**
@@ -86,15 +86,14 @@ public void uninstalled()
         CoherenceApplicationTypeFactory.shutdown();
 
         // un-register the tracer probes
-        if (provider == null)
+        if (m_provider == null)
             {
-            provider = new TracerPackageProviderImpl();
+            m_provider = new TracerPackageProviderImpl();
             }
 
-        TracerSupport.getInstance().unregisterPackageProvider(provider);
+        TracerSupport.getInstance().unregisterPackageProvider(m_provider);
         }
-
-
+
     /**
      * Provider of Coherence tracer probes.
      */
@@ -111,7 +110,7 @@ public TracerPackage<Application>[] getPackages(Application application)
             {
             return new ClusterMonitorPackage[] {new ClusterMonitorPackage(application)};
             }
-    }
+        }
 
-    private TracerPackageProviderImpl provider;
- }
+    private transient TracerPackageProviderImpl m_provider;
+    }

coherence-visualvm-plugin/src/main/java/com/oracle/coherence/plugin/visualvm/tracer/AbstractCoherenceMonitorProbe.java

@@ -58,7 +58,7 @@ public AbstractCoherenceMonitorProbe(int valuesCount, ProbeItemDescriptor[] item
         this.f_resolver     = resolver;
         }
 
-    public synchronized final long[] getItemValues(long timestamp)
+    public final synchronized long[] getItemValues(long timestamp)
         {
         VisualVMModel model = f_resolver.getMonitoredData();
         if (model != null)
@@ -91,7 +91,7 @@ protected long getValueAsLong(Object oValue)
             }
         if (oValue instanceof Integer)
             {
-            return (long)((Integer)oValue);
+            return ((Integer)oValue);
             }
         if (oValue instanceof String)
             {
@@ -126,8 +126,8 @@ protected long[] getSingValue(VisualVMModel model, VisualVMModel.DataType dataTy
 
     private final int f_nValuesCount;
 
-    protected final long[] ZERO_VALUES2 = new long[]{0L,0L};
-    protected final long[] ZERO_VALUES1 = new long[]{0L};
+    protected static final long[] ZERO_VALUES2 = new long[]{0L,0L};
+    protected static final long[] ZERO_VALUES1 = new long[]{0L};
 
     // ----- constants ------------------------------------------------------
 

coherence-visualvm-plugin/src/main/java/com/oracle/coherence/plugin/visualvm/tracer/CustomFormatter.java

@@ -40,30 +40,29 @@ public class CustomFormatter
 
     public CustomFormatter(int factor, String units)
         {
-        this.factor = factor;
-        this.units = units;
+        this.m_nFactor = factor;
+        this.m_sUnits  = units;
         }
 
     @Override
     public String formatValue(long value, int format)
         {
-        return FORMAT.format(value / (double) this.factor);
+        return FORMAT.format(value / (double) this.m_nFactor);
         }
 
     @Override
     public String getUnits(int format)
         {
-        return this.units;
+        return this.m_sUnits;
         }
 
     // ----- constants ------------------------------------------------------
 
     private static final NumberFormat FORMAT = NumberFormat.getInstance();
-
-
+
     // ----- data members ---------------------------------------------------
 
-    private final int    factor;
-    private final String units;
+    private final int    m_nFactor;
+    private final String m_sUnits;
 
     }

pom.xml

@@ -136,7 +136,8 @@
         <artifactId>org-graalvm-visualvm-host</artifactId>
         <version>${visualvm.version}</version>
       </dependency>
-      <!-- trace dependency -->
+
+      <!-- tracer dependency -->
       <dependency>
         <groupId>org.graalvm.visualvm.modules</groupId>
         <artifactId>org-graalvm-visualvm-modules-tracer</artifactId>