Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for custom metrics #322

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

anuraj381
Copy link

Details

As of now the connector only exposes standard Kafka Connect MBeans metrics, but we do have lot of operations happening in connector runtime where we are missing observability. This PR adds support for exposing custom metrics from the connector runtime by adding custom JMX MBeans.

Changes

  • Added MXBean for exporting connector metrics through JMX
  • Since connector Batch mode load-job-runnables do not have observability, added custom metrics to expose numbers around number of active jobs, storage-blobs etc.
  • made custom metrics are easily expandable for metrics around various other elements of the connector

Testing

Tested the exported metrics locally with the Jconsole, I am able to see metrics exported from all tasks of the connector.
Screenshot 2023-05-08 at 12 50 59 AM

Note

  • These custom metrics also facilitates later on adding alerts, for ex. any load-job failures, load-jobs getting triggered in regular intervals etc.
  • code is expandable for adding more custom metrics from connector sink task

* Added MXBean for exporting connector metrics through JMX
* Since connector Batch mode load-job-runnables do not have observability, added custom metrics to expose numbers around number of active jobs, storage-blobs etc.
* made custom metrics are easily expandable for metrics around various other elements of the connector
@anuraj381 anuraj381 requested a review from a team as a code owner May 8, 2023 19:18
@anuraj381
Copy link
Author

@pbadani @b-goyal @aniketshrimal Hi Guys can you please review this PR

@jx2lee
Copy link

jx2lee commented Aug 16, 2024

Will this PR not be reviewed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants