-
Notifications
You must be signed in to change notification settings - Fork 626
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
OTLP exporter should support multi-destination exporting #1109
Comments
With each exporter contained within |
I agree. We can consider using multiple processor each with it's own instance of OTLP exporter as a solution for this. Looks like the spec just recommends having one queue per destination which would be satisfied by such a setup. There's no point in re-implementing BatchSpanProcessor's queue inside the exporter IMO. If we agree with this, this should just be a small documentation change. |
Using multiple exports+processors does probably not technically satisfy the spec as the spec clearly talks about multi-destination exporting in the context of OTLP exporter but still, we can still recommend wrapping multiple OTLP exporters with different processors to achieve the same result. I realize it may not truly satisfy the spec but it addresses the problem the spec is trying to solve in a rather elegant way without adding too much logic/responsibility to the exporter. |
@owais |
This issue was marked stale due to lack of activity. It will be closed in 30 days. |
I somehow missed your comment @lzchen. Yes, I think we should document "how to export to multiple destinations" and it should cover using different exporters or multiple instances of same exporter. |
As per spec: https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/protocol/otlp.md#multi-destination-exporting
The text was updated successfully, but these errors were encountered: