Add Support for Jaeger Kafka Exporter

[joe-elliott]

As discussed here: open-telemetry/opentelemetry-collector#500

Jaeger Kafka exporter support should be added to the collector through the contrib repo.

[yurishkuro]

What is the mechanism used to pull contrib code into the main binary?

[tigrannajaryan]

What is the mechanism used to pull contrib code into the main binary?

@yurishkuro by adding a factory here: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/master/cmd/otelcontribcol/components.go

[pavolloffay]

As an experiment, I have built https://github.com/jaegertracing/jaeger-opentelemetry-collector. At the moment it only includes Jaeger Elasticsearch exporter and the set of the default components. It might be easier to directly build the OTEL collector in the main Jaeger repository - it does not have to be hosted separately.

We should decide where Jaeger storage exporters will live. It depends if we want to add Jaeger specific functionality to the collector. For example one of the missing features is serving sampling strategies.

[joe-elliott]

@pavolloffay
I am working on sampling strategies now to resolve this issue: open-telemetry/opentelemetry-collector#500

[yurishkuro]

It might be easier to directly build the OTEL collector in the main Jaeger repository

+1

[tigrannajaryan]

@yurishkuro @pavolloffay I am happy to see you are building on top of OpenTelemetry Collector. Do you mind sharing a bit more about what you plan to do? It would be great to know how the Collector is used by Jaeger.

[joe-elliott]

Per discussions with Jaeger this functionality is being implemented in https://github.com/jaegertracing/jaeger-opentelemetry-collector and probably does not make sense to implement here as well.

@ledor473, I believe that once jaeger-opentelemetry-collector is up to date with master (for the sampling json) and they add the Kafka exporter this project will meet your needs.

cc @pavolloffay

[pavolloffay]

+1 on what @joe-elliott said. I will just add that in Jaeger we are considering supporting OTEL proto as a first-class citizen. Therefore the contrib repository could provide kafka exporter and in the Jaeger collector there will be an importer for kafka in OT proto format.

[tigrannajaryan]

Per discussions with Jaeger this functionality is being implemented in https://github.com/jaegertracing/jaeger-opentelemetry-collector and probably does not make sense to implement here as well.

Closing this issue as there is nothing else needed to be done in this repo.