Author: veithen
Date: Thu Sep  8 11:24:02 2011
New Revision: 1166634

URL: http://svn.apache.org/viewvc?rev=1166634&view=rev
Log:
Performance improvement: instead of creating a non thread safe SimpleDataFormat 
per connection, use a single thread safe FastDateFormat (from commons-lang).

This results in a measurable increase in throughput (> 10%). Scenario: SOAPUI 
sending with 5 threads to SimpleStockQuoteService (with System.out.println 
statement removed) deployed on Axis2 test server shipped with Synapse (but with 
DDOM used as Axiom implementation; performance increase with standard Axiom 
implementation may be lower).

Modified:
    synapse/trunk/java/modules/transports/core/nhttp/pom.xml
    
synapse/trunk/java/modules/transports/core/nhttp/src/main/java/org/apache/synapse/transport/nhttp/debug/AbstractConnectionDebug.java
    synapse/trunk/java/pom.xml

Modified: synapse/trunk/java/modules/transports/core/nhttp/pom.xml
URL: 
http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/core/nhttp/pom.xml?rev=1166634&r1=1166633&r2=1166634&view=diff
==============================================================================
--- synapse/trunk/java/modules/transports/core/nhttp/pom.xml (original)
+++ synapse/trunk/java/modules/transports/core/nhttp/pom.xml Thu Sep  8 
11:24:02 2011
@@ -147,6 +147,10 @@
             <groupId>org.apache.axis2</groupId>
             <artifactId>axis2-transport-http</artifactId>
         </dependency>
+        <dependency>
+            <groupId>commons-lang</groupId>
+            <artifactId>commons-lang</artifactId>
+        </dependency>
     </dependencies>
 
     <properties>

Modified: 
synapse/trunk/java/modules/transports/core/nhttp/src/main/java/org/apache/synapse/transport/nhttp/debug/AbstractConnectionDebug.java
URL: 
http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/core/nhttp/src/main/java/org/apache/synapse/transport/nhttp/debug/AbstractConnectionDebug.java?rev=1166634&r1=1166633&r2=1166634&view=diff
==============================================================================
--- 
synapse/trunk/java/modules/transports/core/nhttp/src/main/java/org/apache/synapse/transport/nhttp/debug/AbstractConnectionDebug.java
 (original)
+++ 
synapse/trunk/java/modules/transports/core/nhttp/src/main/java/org/apache/synapse/transport/nhttp/debug/AbstractConnectionDebug.java
 Thu Sep  8 11:24:02 2011
@@ -19,10 +19,9 @@
 
 package org.apache.synapse.transport.nhttp.debug;
 
+import org.apache.commons.lang.time.FastDateFormat;
 import org.apache.http.Header;
 
-import java.text.DateFormat;
-import java.text.SimpleDateFormat;
 import java.util.Date;
 import java.util.List;
 
@@ -31,7 +30,7 @@ import java.util.List;
  */
 public abstract class AbstractConnectionDebug {
 
-    protected final DateFormat formatter = new 
SimpleDateFormat("HH:mm:ss.SSS");
+    protected static final FastDateFormat formatter = 
FastDateFormat.getInstance("HH:mm:ss.SSS");
 
     protected String keyValueSeparator;
     protected String fieldSeparator;

Modified: synapse/trunk/java/pom.xml
URL: 
http://svn.apache.org/viewvc/synapse/trunk/java/pom.xml?rev=1166634&r1=1166633&r2=1166634&view=diff
==============================================================================
--- synapse/trunk/java/pom.xml (original)
+++ synapse/trunk/java/pom.xml Thu Sep  8 11:24:02 2011
@@ -459,6 +459,11 @@
                 <artifactId>commons-pool</artifactId>
                 <version>${commons.pool.version}</version>
             </dependency>
+            <dependency>
+                <groupId>commons-lang</groupId>
+                <artifactId>commons-lang</artifactId>
+                <version>${commons-lang.version}</version>
+            </dependency>
 
             <dependency>
                 <groupId>log4j</groupId>
@@ -664,7 +669,6 @@
         <dependency>
             <groupId>commons-lang</groupId>
             <artifactId>commons-lang</artifactId>
-            <version>${commons-lang.version}</version>
             <scope>test</scope>
         </dependency>
         


Reply via email to