newrelic / kafka-connect-newrelic

Apache License 2.0
5 stars 17 forks source link

[SMT] Creates a new event rollup to kafka connect new relic #32

Closed wolvery closed 3 years ago

wolvery commented 3 years ago

Hi,

I have created this PR to make possible to a different kind of event from the available "agent" could be uploaded to New Relic. This has been tested with Kafka Connect base 6.0.1.

I hope it can get some reviews and be part of the project.

Thank you very much, Guilherme

CLAassistant commented 3 years ago

CLA assistant check
All committers have signed the CLA.

JimHagan commented 3 years ago

@wolvery Can you give a brief description of the use case that was driving this PR? Thanks.

wolvery commented 3 years ago

@JimHagan, we are using this connector to export events from an avro topic to new relic. We emit events when some problem occurs and we need them at New Relic to alert

ericmittelhammer commented 3 years ago

Hi @wolvery - thanks for the PR. The changes are are in the right sprit. However, there is currently an open PR for v2.0 which I believe would handle both of your use cases (Avro messages, and processing single events without having to use an SMT to insert them into a list).

wolvery commented 3 years ago

Well, in this case, we can close this PR. I look forward to this new version! Thank you very much.

ericmittelhammer commented 3 years ago

Closing, as functionality is inclusive in 2.0.0