search

newrelic / kafka-connect-newrelic

Apache License 2.0
5 stars 17 forks source link

General code and test coverage improvements; Version Upgrades; Fixed batch sender bugs #30

Closed amoswood closed 3 years ago

amoswood commented 3 years ago

Items which I accomplished in this update:

  1. Reorganized tests to proper packages
  2. Improved test coverage and added the missing classes
  3. Updated the New Relic telemetry API to 0.10.0
  4. Updated the Kafka version to 2.7.0
  5. Fixed batch sender bugs in the metrics and events code when the batch would never be cleared and continue to get sent with each new message causing a ton of duplicates in New Relic.
CLAassistant commented 3 years ago

CLA assistant check
All committers have signed the CLA.

amoswood commented 3 years ago

@rahulbats , What do you think of this? We have been using it for our testing and now when moving data to production with great success. I would suggest that it get merged into the code base and then deployed to Confluent hub.

JimHagan commented 3 years ago

@amoswood Please resolve conflicts if possible.

amoswood commented 3 years ago

After the upgrade, there are no needed changes from the base version.