Skip to content

KC-1 Document new metrics for Inkless - #592

Merged
ivanyu merged 4 commits into
mainfrom
KC-1-metrics-docs
May 13, 2026
Merged

KC-1 Document new metrics for Inkless#592
ivanyu merged 4 commits into
mainfrom
KC-1-metrics-docs

Conversation

@jlprat

@jlprat jlprat commented May 12, 2026

Copy link
Copy Markdown
Member

Adds documentation to the new metrics added
Enhances the MetricsDoc class to read from all the metric providers

Adds documentation to the new metrics added
Enhances the MetricsDoc class to read from all the
metric providers

Signed-off-by: Josep Prat <josep.prat@aiven.io>
Comment thread storage/inkless/src/main/java/io/aiven/inkless/produce/WriterMetrics.java Outdated
Comment thread storage/inkless/src/main/java/io/aiven/inkless/produce/WriterMetrics.java Outdated
Signed-off-by: Josep Prat <josep.prat@aiven.io>
@jlprat

jlprat commented May 13, 2026

Copy link
Copy Markdown
Member Author

Thanks @ivanyu applied the feedback

Signed-off-by: Josep Prat <josep.prat@aiven.io>
ivanyu
ivanyu previously approved these changes May 13, 2026
Signed-off-by: Josep Prat <josep.prat@aiven.io>
@ivanyu
ivanyu enabled auto-merge (squash) May 13, 2026 08:57
@ivanyu
ivanyu merged commit cfd86aa into main May 13, 2026
4 checks passed
@ivanyu
ivanyu deleted the KC-1-metrics-docs branch May 13, 2026 08:57
giuseppelillo pushed a commit that referenced this pull request May 15, 2026
* KC-1 Document new metrics for Inkless

Adds documentation to the new metrics added
Enhances the MetricsDoc class to read from all the
metric providers

Signed-off-by: Josep Prat <josep.prat@aiven.io>

* Apply feedback

Signed-off-by: Josep Prat <josep.prat@aiven.io>

* Fix import order

Signed-off-by: Josep Prat <josep.prat@aiven.io>

* Add updated doc file

Signed-off-by: Josep Prat <josep.prat@aiven.io>

---------

Signed-off-by: Josep Prat <josep.prat@aiven.io>
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