Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enable tracing in DistributedCommandBus with SpanFactory #2403

Closed
smcvb opened this issue Sep 27, 2022 · 1 comment · Fixed by #2729
Closed

Enable tracing in DistributedCommandBus with SpanFactory #2403

smcvb opened this issue Sep 27, 2022 · 1 comment · Fixed by #2729
Assignees
@gklijs
Labels
Priority 2: Should High priority. Ideally, these issues are part of the release they’re assigned to. Status: Resolved Use to signal that work on this issue is done. Type: Feature Use to signal an issue is completely new to the project.
Milestone
Release 4.8.0

Comments

@smcvb
Copy link
Member

smcvb commented Sep 27, 2022

Feature Description

Axon Framework 4.6.0 has seen the introduction of integrated tracing through issue #2294.
This issue did not cover the DistributedCommandBus, and it's CommandRouter and CommandBusConnector, though.

Current Behaviour

The integrated tracing support is not in place for the DistributedCommandBus.

Wanted Behaviour

Introduce the SpanFactory in the DistributedCommandBus to trace it's processes.
@smcvb smcvb added Type: Feature Use to signal an issue is completely new to the project. Priority 2: Should High priority. Ideally, these issues are part of the release they’re assigned to. labels Sep 27, 2022
@smcvb smcvb added this to the Release 4.7.0 milestone Sep 27, 2022
@smcvb smcvb modified the milestones: Release 4.7.0, Release 4.8.0 Jan 16, 2023
@smcvb
Copy link
Member Author

smcvb commented Jan 16, 2023

Moving issue to release 4.8.0 in favor of time we need to spent on releasing 4.7.0

@gklijs gklijs self-assigned this May 25, 2023
@gklijs gklijs added the Status: In Progress Use to signal this issue is actively worked on. label May 25, 2023
@gklijs gklijs mentioned this issue May 25, 2023
Merged
gklijs added a commit that referenced this issue Jun 1, 2023
@gklijs
Merge pull request #2729 from /issues/2403
85ef86d 
Enable tracing in DistributedCommandBus with SpanFactory
@gklijs gklijs added Status: Resolved Use to signal that work on this issue is done. and removed Status: In Progress Use to signal this issue is actively worked on. labels Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gklijs gklijs

Labels
Priority 2: Should High priority. Ideally, these issues are part of the release they’re assigned to. Status: Resolved Use to signal that work on this issue is done. Type: Feature Use to signal an issue is completely new to the project.
Projects
None yet
Milestone
Release 4.8.0
Development

Successfully merging a pull request may close this issue.

Enable tracing in DistributedCommandBus with SpanFactory AxonFramework/AxonFramework
3 participants
@gklijs @smcvb @YvonneCeelie