rubenvp8510 commented on pull request #4016:
URL: https://github.com/apache/camel/pull/4016#issuecomment-661043003


   > Please run with the sourcecheck profile enabled because the indentation in 
parent pom seems to be wrong and in the components you'll get a bunch of errors 
to fix in terms of codestyle.
   > 
   > Also, what is the purpose of this PR? I see a camel-tracing component 
introduced. This seems to be a big change and refactoring so it probably 
deserve a JIRA issue with some explanation of what is the plan and what will be 
in the common module and what not.
   > 
   > Thanks.
   
   Hi, Thanks for the review, the purpose is to support opentelemetry, doing 
the same as opentracing does but using opentelemetry instead.
   
   The purpose of the refactorization is for reuse the span decorators. I'll 
check the code style, but I ran the `mvn clean install -Psourcecheck `locally 
and it passed. :/


----------------------------------------------------------------
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.

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


Reply via email to