Author: heiko.braun(a)jboss.com Date: 2009-11-24 10:14:06 -0500 (Tue, 24 Nov 2009) New Revision: 315 Modified: trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/engine/ode/BPELEngineImpl.java trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/ws/SOAPMessageAdapter.java Log: Don't waste CPU cycles Modified: trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/engine/ode/BPELEngineImpl.java =================================================================== --- trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/engine/ode/BPELEngineImpl.java 2009-11-24 15:13:34 UTC (rev 314) +++ trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/engine/ode/BPELEngineImpl.java 2009-11-24 15:14:06 UTC (rev 315) @@ -221,7 +221,7 @@ String messageId = new GUID().toString(); MyRoleMessageExchange odeMex = _bpelServer.getEngine() .createMessageExchange(messageId, adapter.getServiceName(), adapter.getOperationName()); - __log.debug("ODE routed to operation " + odeMex.getOperation() + " from service " + adapter.getServiceName()); + if (__log.isDebugEnabled()) __log.debug("ODE routed to operation " + odeMex.getOperation() + " from service " + adapter.getServiceName()); return odeMex; } Modified: trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/ws/SOAPMessageAdapter.java =================================================================== --- trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/ws/SOAPMessageAdapter.java 2009-11-24 15:13:34 UTC (rev 314) +++ trunk/runtime/engine/src/main/java/org/jboss/soa/bpel/runtime/ws/SOAPMessageAdapter.java 2009-11-24 15:14:06 UTC (rev 315) @@ -35,9 +35,6 @@ import javax.wsdl.extensions.ElementExtensible; import javax.wsdl.extensions.soap.SOAPBinding; import javax.xml.namespace.QName; -import javax.xml.parsers.DocumentBuilder; -import javax.xml.parsers.DocumentBuilderFactory; -import javax.xml.parsers.ParserConfigurationException; import javax.xml.soap.*; import java.util.*; @@ -339,7 +336,7 @@ if (srcPart == null) throw new RuntimeException("Soap body does not contain required part +"+part.getName()); - odeMessage.setPart(srcPart.getLocalName(), cloneElement(srcPart)); + odeMessage.setPart(srcPart.getLocalName(), srcPart); } } else @@ -353,7 +350,7 @@ Document doc = DOMUtils.newDocument(); Element destPart = doc.createElementNS(null, part.getName()); destPart.appendChild(doc.importNode(srcPart, true)); - odeMessage.setPart(part.getName(), cloneElement(destPart)); + odeMessage.setPart(part.getName(), destPart); } } } @@ -374,7 +371,7 @@ return match; } - private static Element cloneElement(Element source) + /*private static Element cloneElement(Element source) { // TODO: https://jira.jboss.org/jira/browse/RIFTSAW-38 // For now create a deep copy (performance hit) @@ -388,7 +385,7 @@ { throw new RuntimeException(e); } - } + } */ public static <T> T getFirstExtensibilityElement(ElementExtensible parent, Class<T> cls) { Collection<T> ee = CollectionsX.filter(parent.getExtensibilityElements(), cls);