oscerd commented on PR #9389: URL: https://github.com/apache/camel/pull/9389#issuecomment-1447766741
> A list of questions and TODOs related to this component that we must resolve before merging > > Discuss: > > * The `Tracer` abstraction from Camel has a notion of only high cardinality tags, we are missing low cardinality tags for metrics (that means that metrics will not have any tags at this point) > * Are the changes in the `Tracer` in the `tracing` component acceptable? > > TODO: > > * [ ] Spring Boot Starter is missing This is not a problem, we could do that later > * [ ] Polish the documentation This could be done with a subsequent PR > * [ ] Currently the behaviour is almost identical to OpenTelemetry one, that requires the presence of custom handlers - I'm considering to remove them so that the additional code will not be necessary. That will mean that with Micrometer Observation with OTel Tracer the spans will be a little bit different than with the OpenTelemetry component On this, I'm not really sure about it: if we goes this way it's important to document and notes the difference, otherwise users might think the result will be the same with Micrometer and Opentelemetry and it won't be the case > Recrated from #9377 -- 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-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
