[teiid-commits] teiid SVN: r1248 - in trunk: connector-api/src/main/java/org/teiid/connector/visitor/util and 3 other directories.

[teiid-commits at lists.jboss.org]

Author: rareddy
Date: 2009-08-17 11:57:48 -0400 (Mon, 17 Aug 2009)
New Revision: 1248

Modified:
   trunk/build/kit-runtime/deploy/jboss-cache-configuration.xml
   trunk/connector-api/src/main/java/org/teiid/connector/visitor/util/SQLStringVisitor.java
   trunk/connectors/connector-jdbc/src/main/java/org/teiid/connector/jdbc/translator/SQLConversionVisitor.java
   trunk/engine/src/main/java/org/teiid/dqp/internal/datamgr/impl/ConnectorManager.java
   trunk/engine/src/main/java/org/teiid/dqp/internal/process/DQPCore.java
Log:
TEIID-786: increased the eviction thread wake up time, avoid making couple expensive logging calls, converted StringBuffer to StringBuilder to remove heavy use of sychronous append calls

Modified: trunk/build/kit-runtime/deploy/jboss-cache-configuration.xml
===================================================================
--- trunk/build/kit-runtime/deploy/jboss-cache-configuration.xml	2009-08-17 15:46:53 UTC (rev 1247)
+++ trunk/build/kit-runtime/deploy/jboss-cache-configuration.xml	2009-08-17 15:57:48 UTC (rev 1248)
@@ -12,7 +12,7 @@
         </loader>
     </loaders>
     
-    <eviction wakeUpInterval="3000">
+    <eviction wakeUpInterval="15000">
         <default algorithmClass="org.jboss.cache.eviction.LRUAlgorithm" eventQueueSize="100000">
             <property name="maxNodes" value="10000" />
             <!-- 0 = immediate eviction, -1 = no limit -->

Modified: trunk/connector-api/src/main/java/org/teiid/connector/visitor/util/SQLStringVisitor.java
===================================================================
--- trunk/connector-api/src/main/java/org/teiid/connector/visitor/util/SQLStringVisitor.java	2009-08-17 15:46:53 UTC (rev 1247)
+++ trunk/connector-api/src/main/java/org/teiid/connector/visitor/util/SQLStringVisitor.java	2009-08-17 15:57:48 UTC (rev 1248)
@@ -92,7 +92,7 @@
     protected static final String UNDEFINED = "<undefined>"; //$NON-NLS-1$
     protected static final String UNDEFINED_PARAM = "?"; //$NON-NLS-1$
     
-    protected StringBuffer buffer = new StringBuffer();
+    protected StringBuilder buffer = new StringBuilder();
                 
     /**
      * Gets the name of a group or element from the RuntimeMetadata

Modified: trunk/connectors/connector-jdbc/src/main/java/org/teiid/connector/jdbc/translator/SQLConversionVisitor.java
===================================================================
--- trunk/connectors/connector-jdbc/src/main/java/org/teiid/connector/jdbc/translator/SQLConversionVisitor.java	2009-08-17 15:46:53 UTC (rev 1247)
+++ trunk/connectors/connector-jdbc/src/main/java/org/teiid/connector/jdbc/translator/SQLConversionVisitor.java	2009-08-17 15:57:48 UTC (rev 1248)
@@ -143,7 +143,7 @@
      * @param object
      * @param valuesbuffer
      */
-    private void translateSQLType(Class type, Object obj, StringBuffer valuesbuffer) {
+    private void translateSQLType(Class type, Object obj, StringBuilder valuesbuffer) {
         if (obj == null) {
             valuesbuffer.append(NULL);
         } else {

Modified: trunk/engine/src/main/java/org/teiid/dqp/internal/datamgr/impl/ConnectorManager.java
===================================================================
--- trunk/engine/src/main/java/org/teiid/dqp/internal/datamgr/impl/ConnectorManager.java	2009-08-17 15:46:53 UTC (rev 1247)
+++ trunk/engine/src/main/java/org/teiid/dqp/internal/datamgr/impl/ConnectorManager.java	2009-08-17 15:57:48 UTC (rev 1248)
@@ -603,6 +603,9 @@
      * @param qr Request that contains the MetaMatrix command information in the transaction.
      */
     void logSRCCommand(AtomicRequestMessage qr, ExecutionContext context, short cmdStatus, int finalRowCnt) {
+    	if (!LogManager.isMessageToBeRecorded(LogConstants.CTX_COMMANDLOGGING, MessageLevel.INFO)) {
+    		return;
+    	}
         String sqlStr = null;
         if(cmdStatus == CommandLogMessage.CMD_STATUS_NEW){
         	Command cmd = qr.getCommand();

Modified: trunk/engine/src/main/java/org/teiid/dqp/internal/process/DQPCore.java
===================================================================
--- trunk/engine/src/main/java/org/teiid/dqp/internal/process/DQPCore.java	2009-08-17 15:46:53 UTC (rev 1247)
+++ trunk/engine/src/main/java/org/teiid/dqp/internal/process/DQPCore.java	2009-08-17 15:57:48 UTC (rev 1248)
@@ -505,6 +505,10 @@
 	}
     
     void logMMCommand(RequestWorkItem workItem, boolean isBegin, boolean isCancel, int rowCount) {
+    	if (!LogManager.isMessageToBeRecorded(LogConstants.CTX_COMMANDLOGGING, MessageLevel.INFO)) {
+    		return;
+    	}
+    	
         RequestMessage msg = workItem.requestMsg;
         DQPWorkContext workContext = DQPWorkContext.getWorkContext();
         RequestID rID = new RequestID(workContext.getConnectionID(), msg.getExecutionId());