You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ni...@apache.org on 2014/09/26 08:05:43 UTC

[1/2] git commit: CAMEL-7863: Improve logging message and reduce verbosity from INFO to debug

Repository: camel
Updated Branches:
  refs/heads/master b1e84ee9b -> 62f08f081


CAMEL-7863: Improve logging message and reduce verbosity from INFO to debug


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/37250ba0
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/37250ba0
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/37250ba0

Branch: refs/heads/master
Commit: 37250ba0283b86ad61fde0c331097d6867acd2c9
Parents: b1e84ee
Author: david <da...@davidkarlsen.com>
Authored: Thu Sep 25 19:57:06 2014 +0200
Committer: Willem Jiang <wi...@gmail.com>
Committed: Fri Sep 26 10:38:28 2014 +0800

----------------------------------------------------------------------
 .../main/java/org/apache/camel/component/cxf/CxfProducer.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/37250ba0/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
----------------------------------------------------------------------
diff --git a/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java b/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
index 70df6e2..51520ad 100644
--- a/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
+++ b/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
@@ -336,11 +336,11 @@ public class CxfProducer extends DefaultProducer implements AsyncProcessor {
         BindingOperationInfo answer = null;
         String lp = ex.getIn().getHeader(CxfConstants.OPERATION_NAME, String.class);
         if (lp == null) {
-            LOG.info("CxfProducer cannot find the {} from message header, try to use the defaultOperationName", CxfConstants.OPERATION_NAME);
+            LOG.debug("CxfProducer cannot find the {} from message header, trying with defaultOperationName", CxfConstants.OPERATION_NAME);
             lp = endpoint.getDefaultOperationName();
         }
         if (lp == null) {
-            LOG.info("CxfProducer cannot find the {} from message header and there is no DefaultOperationName setting, CxfProducer will pick up the first available operation.",
+            LOG.debug("CxfProducer cannot find the {} from message header and there is no DefaultOperationName setting, CxfProducer will pick up the first available operation.",
                      CxfConstants.OPERATION_NAME);
             Collection<BindingOperationInfo> bois = 
                 client.getEndpoint().getEndpointInfo().getBinding().getOperations();


[2/2] git commit: CAMEL-7863 Updated the java doc for the operation name lookup.

Posted by ni...@apache.org.
CAMEL-7863 Updated the java doc for the operation name lookup.


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/62f08f08
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/62f08f08
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/62f08f08

Branch: refs/heads/master
Commit: 62f08f0813a3e82a02a35925c0ac8844bfb8e887
Parents: 37250ba
Author: Willem Jiang <wi...@gmail.com>
Authored: Fri Sep 26 11:29:36 2014 +0800
Committer: Willem Jiang <wi...@gmail.com>
Committed: Fri Sep 26 11:29:36 2014 +0800

----------------------------------------------------------------------
 .../java/org/apache/camel/component/cxf/CxfProducer.java  | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/62f08f08/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
----------------------------------------------------------------------
diff --git a/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java b/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
index 51520ad..0aa4fa9 100644
--- a/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
+++ b/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfProducer.java
@@ -328,8 +328,14 @@ public class CxfProducer extends DefaultProducer implements AsyncProcessor {
     }
 
     /**
-     * Get operation name from header and use it to lookup and return a 
-     * {@link BindingOperationInfo}.
+     * <p>Get operation name from header and use it to lookup and return a 
+     * {@link BindingOperationInfo}.</p>
+     * <p>CxfProducer lookups the operation name lookup with below order, and it uses the first found one which is not null:</p>
+     *  <ul>
+     *    <li> Using the in message header "operationName". </li>
+     *    <li> Using the defaultOperationName option value from the CxfEndpoint. </li>
+     *    <li> Using the first operation which is find from the CxfEndpoint Operations list. </li>
+     *  <ul>
      */
     private BindingOperationInfo getBindingOperationInfo(Exchange ex) {
         CxfEndpoint endpoint = (CxfEndpoint)this.getEndpoint();