You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "mcarlett (via GitHub)" <gi...@apache.org> on 2024/08/26 08:20:23 UTC

[PR] CAMEL-21118: Adds rest-cxf-opentelemetry [camel-spring-boot-examples]

mcarlett opened a new pull request, #142:
URL: https://github.com/apache/camel-spring-boot-examples/pull/142

   (no comment)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] CAMEL-21118: Adds rest-cxf-opentelemetry [camel-spring-boot-examples]

Posted by "mcarlett (via GitHub)" <gi...@apache.org>.
mcarlett commented on PR #142:
URL: https://github.com/apache/camel-spring-boot-examples/pull/142#issuecomment-2309729951

   yes exactly, the target here is to show how to use OTEL instrumentation and CXF in CSB, related to the REST services, I try to improve the example and the readme as well, thanks @davsclaus for the feedback


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] CAMEL-21118: Adds rest-cxf-opentelemetry [camel-spring-boot-examples]

Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus merged PR #142:
URL: https://github.com/apache/camel-spring-boot-examples/pull/142


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] CAMEL-21118: Adds rest-cxf-opentelemetry [camel-spring-boot-examples]

Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #142:
URL: https://github.com/apache/camel-spring-boot-examples/pull/142#issuecomment-2309712815

   When working with SOAP you usually have a WSDL contract and then source code generate the JAXB model classes.
   And then camel-jaxb can automatic convert to/from these classes.
   
   This example uses a code-first approach which is only for hello-world and very small web services.
   This is not used in real world use-cases - only for small "hello world".
   
   I understand the examples wants to focus and show OTEL with CXF. And as such this is less of a problem, but should IMHO be explained in the readme the reason for this. 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org