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


Reply via email to