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.

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

Open telemetry applied #181

Merged
merged 12 commits into from Nov 26, 2022
Merged

Open telemetry applied #181

merged 12 commits into from Nov 26, 2022

Conversation

oskardudycz
Copy link
Owner

@oskardudycz oskardudycz commented Nov 18, 2022

Introduced Open Telemetry tracing together with context propagation:

  • CQRS processing,
  • Marten,
  • EventStoreDB,
  • Kafka.

Removed MediatR as:

  • I'd like to have more customisation around Open Telemetry instrumentation,
  • Retries,
  • I haven't used either pipelines or behaviours,
  • it was more limiting than helping (especially with its enforcement of marker interfaces).

Removed previous manual instrumentation around correlation and causation ids.

TODO:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant