opentelemetry-demo icon indicating copy to clipboard operation
opentelemetry-demo copied to clipboard

Implement message queue

Open secustor opened this issue 1 year ago • 1 comments

Fixes #429

Changes

  • [x] add Kafka to docker-compose
  • [x] add Kafka producer to checkoutservice
  • [x] implement Accountingservice
  • [x] implement Frauddetectionservice

For significant contributions please make sure you have completed the following items:

  • [x] Appropriate CHANGELOG.md updated for non-trivial changes
  • [x] Design discussion issue #429

secustor avatar Oct 17 '22 11:10 secustor

Hey just an fyi we're on a feature freeze for the next week or 2 due to our upcoming v1 release and kubecon.

Feel free to keep this open and working on it tho!

cartersocha avatar Oct 19 '22 00:10 cartersocha

I wonder that why add two new services? Since the checkout service is the only kafka message producer, the two new services are both consumers. What's the point of this?

fatsheep9146 avatar Oct 31 '22 13:10 fatsheep9146

Async workflows usually have multiple consumers with different use cases.
The intention is to show async with OTEL in a common scenario and that both spans are added to the parent span of the checkoutservice

secustor avatar Oct 31 '22 15:10 secustor

This PR was marked stale due to lack of activity. It will be closed in 7 days.

github-actions[bot] avatar Nov 10 '22 03:11 github-actions[bot]