feat: add opentelemetry-exporter-http-transport package#5194
Conversation
|
|
||
| Core package (no HTTP backend included):: | ||
|
|
||
| pip install opentelemetry-exporter-http-transport |
There was a problem hiding this comment.
If this is specific to otlp over http, should the package name reflect that: opentelemetry-exporter-otlp-http-transport
There was a problem hiding this comment.
I was actually debating this exact naming decision, I decided that opentelemetry-exporter-http-transport might be slightly more appropriate than opentelemetry-exporter-otlp-http-transport given that we might want to use the transport abstraction for non-OTLP exporters (e.g. Jaeger). That being said, I still opted to include the OTLPHTTPClient inside this package since it's pretty lean and will be reused by the Proto + JSON HTTP exporters, but we could even break this off into another package, but my thought is this would be a bit overkill. WDYT?
Description
Adds a common
opentelemetry-exporter-http-transportpackage that will be utilized by multiple HTTP exporter packages (initially OTLP JSON) to abstract away and generalize underlying HTTP transport implementations and exporting with OTLP. Introducing this package will significantly reduce the amount of code duplication present in HTTP exporters and will allow for the usage of alternative HTTP client libraries.Fixes #3439, #4171, #1003, #2990
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Does This PR Require a Contrib Repo Change?
Checklist: