You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@synapse.apache.org by hi...@apache.org on 2010/01/05 06:20:25 UTC

svn commit: r895908 - in /synapse/trunk/java/modules/core/src/main/java/org/apache/synapse: SynapseConstants.java endpoints/AbstractEndpoint.java

Author: hiranya
Date: Tue Jan  5 05:20:24 2010
New Revision: 895908

URL: http://svn.apache.org/viewvc?rev=895908&view=rev
Log:
Adding a new constant to replace the hardcoded string used in endpoint stat generation logic


Modified:
    synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/SynapseConstants.java
    synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/AbstractEndpoint.java

Modified: synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/SynapseConstants.java
URL: http://svn.apache.org/viewvc/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/SynapseConstants.java?rev=895908&r1=895907&r2=895908&view=diff
==============================================================================
--- synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/SynapseConstants.java (original)
+++ synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/SynapseConstants.java Tue Jan  5 05:20:24 2010
@@ -282,6 +282,9 @@
     /** A name to use for anonymous sequences in the sequence stack */
     public static final String ANONYMOUS_SEQUENCE = "AnonymousSequence";
 
+    /** String to be used as the separator when defining resource IDs for statistics */
+    public static final String STATISTICS_KEY_SEPARATOR = "__";
+
     /** Message format values in EndpointDefinition. Used by address, wsdl endpoints */
     public static final String FORMAT_POX = "pox";
     public static final String FORMAT_GET = "get";

Modified: synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/AbstractEndpoint.java
URL: http://svn.apache.org/viewvc/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/AbstractEndpoint.java?rev=895908&r1=895907&r2=895908&view=diff
==============================================================================
--- synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/AbstractEndpoint.java (original)
+++ synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/AbstractEndpoint.java Tue Jan  5 05:20:24 2010
@@ -383,7 +383,8 @@
             if (opName != null) {
                 if (oldConfiguration.isStatisticsEnable()) {
                     AspectConfiguration newConfiguration = new AspectConfiguration(
-                            oldConfiguration.getId() + "." + opName);
+                            oldConfiguration.getId() + SynapseConstants.STATISTICS_KEY_SEPARATOR +
+                                    opName);
                     newConfiguration.enableStatistics();
                     StatisticsReporter.reportForComponent(synCtx, newConfiguration,
                             ComponentType.ENDPOINT);