Skip to content
This repository has been archived by the owner on Sep 14, 2022. It is now read-only.

Add PRODUCER and CONSUMER span kinds #104

Open
joaoandremartins opened this issue Apr 27, 2018 · 0 comments
Open

Add PRODUCER and CONSUMER span kinds #104

joaoandremartins opened this issue Apr 27, 2018 · 0 comments

Comments

@joaoandremartins
Copy link

Not sure if this is the correct place to file this issue. Please let me know if I should be knocking at someone else's door.

Besides CLIENT and SERVER span types, Zipkin also contemplates PRODUCER and CONSUMER spans.

For example, for PRODUCER, JavaDoc says:

/**
* When present, {@link #timestamp()} is the moment a producer sent a message to a destination.
* {@link #duration()} represents delay sending the message, such as batching, while {@link
* #remoteEndpoint()} indicates the destination, such as a broker.
*
*

Unlike {@link #CLIENT}, messaging spans never share a span ID. For example, the {@link
* #CONSUMER} of the same message has {@link #parentId()} set to this span's {@link #id()}.

In Spring Cloud GCP, we integrate Zipkin with Stackdriver Trace, so traces processed by Zipkin are stored in Stackdriver Trace.

We do a mapping of Zipkin to Stackdriver Trace span kind, but Stackdriver Trace doesn't have an obvious mapping for PRODUCER and CONSUMER.

Would it make sense to add them to Stackdriver Trace as well?

My gut feeling is that, because PRODUCER and CONSUMER don't share their trace ID with any spans, it can probably left out as UNSPECIFIED, but would like to double check this with the Stackdriver Trace team.

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

No branches or pull requests

1 participant