Skip to content

Added support for custom metrics#322

Open
Anuraj Jain (anuraj381) wants to merge 2 commits into
confluentinc:masterfrom
anuraj381:custom_metrics
Open

Added support for custom metrics#322
Anuraj Jain (anuraj381) wants to merge 2 commits into
confluentinc:masterfrom
anuraj381:custom_metrics

Conversation

@anuraj381
Copy link
Copy Markdown

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 Anuraj Jain (anuraj381) requested a review from a team as a code owner May 8, 2023 19:18
@anuraj381
Copy link
Copy Markdown
Author

Parag Badani (@pbadani) Bhagyashree (@b-goyal) Aniket Shrimal (@aniketshrimal) Hi Guys can you please review this PR

@jx2lee
Copy link
Copy Markdown

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