Skip to content

Conversation

@juha-aiven
Copy link
Contributor

@juha-aiven juha-aiven commented Jul 11, 2025

In order to expose the metrics, refactor WriterMetrics. For each metric introduce a corresponding MetricName. Add the description constants for all metrics. Finally, expose the metrics as MetricNameTemplate.

@juha-aiven juha-aiven marked this pull request as draft July 11, 2025 05:52
Add Gradle task :storage:inkless:genInklessMetricsDoc that generates the file docs/inkless/metrics.rst documenting Inkless-specific metrics. The task executes aiven.inkless.doc.MetricsDocs.

At this point only AWS S3 storage backend metrics, Azure storage backend metrics and Google Cloud Storage backend metrics are included.
@juha-aiven juha-aiven force-pushed the metrics-documentation-storage-backends branch from 062556f to d43b853 Compare July 11, 2025 10:22
In order to expose the metrics, refactor WriterMetrics. For each metric introduce a corresponding MetricName. Add the description constants for all metrics. Finally, expose the metrics as MetricNameTemplate.
@juha-aiven juha-aiven force-pushed the metrics-documentation-inkless-writer-and-reader branch from eabde76 to f0d30e7 Compare July 11, 2025 11:10
@juha-aiven juha-aiven marked this pull request as ready for review July 11, 2025 13:45
Base automatically changed from metrics-documentation-storage-backends to main July 11, 2025 15:24
@github-actions
Copy link

This PR is being marked as stale since it has not had any activity in 90 days. If you
would like to keep this PR alive, please leave a comment asking for a review. If the PR has
merge conflicts, update it with the latest from the base branch.

If you are having difficulty finding a reviewer, please reach out on the [mailing list](https://kafka.apache.org/contact).

If this PR is no longer valid or desired, please feel free to close it. If no activity occurs in the next 30 days, it will be automatically closed.

@github-actions github-actions bot added the stale label Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants