New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable tracing in DistributedCommandBus with SpanFactory #2729
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The premise looks good, but there is a small issue with it. Also, there is some javadoc missing.
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Outdated
Show resolved
Hide resolved
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Outdated
Show resolved
Hide resolved
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of comments do resolve before we proceed.
Furthermore, it is good to point out this resolves the sending side for tracing, but not the handling end. For setting the handle span, we need to adjust the CommandBusConnector
for the Spring Cloud and JGroups extension. Not necessarily a task you should do ASAP, but we shouldn't forget, of course!
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Show resolved
Hide resolved
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Show resolved
Hide resolved
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Show resolved
Hide resolved
...g/src/test/java/org/axonframework/commandhandling/distributed/DistributedCommandBusTest.java
Show resolved
Hide resolved
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Show resolved
Hide resolved
...aging/src/main/java/org/axonframework/commandhandling/distributed/DistributedCommandBus.java
Outdated
Show resolved
Hide resolved
…sing the spanfactory to propagate the context.
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My concerns have been addressed, hence I'm approving this pull request.
fixes #2403
I think all the needed tracing is there, but I'm unsure.