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

add an online parser for prometheus. #2851

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

Conversation

leo-934
Copy link
Contributor

@leo-934 leo-934 commented Dec 1, 2024

add an online parser for prometheus, which replaces org.apache.hertzbeat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies

What's changed?

Checklist

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
@leo-934 leo-934 force-pushed the online-prometheus-parser branch from 7c2f8b5 to e93e161 Compare December 1, 2024 17:47
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
…eat.collector.collect.prometheus.parser.TextParser.textToMetricFamilies
@tomsun28 tomsun28 requested a review from Ceilzcx December 4, 2024 11:13
@tomsun28 tomsun28 added this to the 1.6.2 milestone Dec 4, 2024
@tomsun28 tomsun28 added the good first pull request Good for newcomers label Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

4 participants