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

Add Influx Client as Metric Exporter

Open MariusBrill opened this issue 3 years ago • 5 comments

Is your feature request related to a problem? Please describe. Currently OpenTelemetry does not provide a default InfluxDB Exporter. Since we would like to add an InfluxDB Exporter to the inspectIT Ocelot, we would like to contribute to Open Telemetry and add this exporter to the project.

Describe the solution you'd like The exporter would use the Influx Java Client and would integrate it similar to the OTLP Metric Exporter to ensure a seamless integration into OTLP.

Additional context Could you guys give me short feedback if this would fit for you into the project? I am open to discuss changes/alternatives.

Best Regards Marius

MariusBrill avatar Jul 12 '22 09:07 MariusBrill

I think this is a great idea, but until there's a official specification for this, it would need to live in the contrib repo, rather than the core repo.

jkwatson avatar Jul 12 '22 20:07 jkwatson

Hey @jkwatson, I just opened the corresponding issue in the contrib repo. Thanks alot!

Greetings Marius

MariusBrill avatar Jul 19 '22 06:07 MariusBrill

Hey @jkwatson, I just opened the corresponding issue in the contrib repo. Thanks alot!

Greetings Marius

FYI I have a partially working version of this in a personal project of mine and would love to use a more official one. :)

jkwatson avatar Jul 19 '22 14:07 jkwatson

Hi @jkwatson,

sounds cool! It's always awesome to contribute features others too see a need in :) Are there any learnings or pitfalls you encountered that i should be aware of during development?

MariusBrill avatar Jul 20 '22 06:07 MariusBrill

Hi @jkwatson,

sounds cool! It's always awesome to contribute features others too see a need in :) Are there any learnings or pitfalls you encountered that i should be aware of during development?

Not really. My personal project only uses gauges, so I only implemented what was the absolute minimum to support that. :)

jkwatson avatar Jul 20 '22 14:07 jkwatson

Closing in favor of open-telemetry/opentelemetry-java-contrib#394.

jack-berg avatar Oct 25 '22 20:10 jack-berg