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

[BUG] Error thrown when parsing local index data #204

Closed
ansjcy opened this issue Jan 24, 2025 · 0 comments
Closed

[BUG] Error thrown when parsing local index data #204

ansjcy opened this issue Jan 24, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@ansjcy
Copy link
Member

ansjcy commented Jan 24, 2025

What is the bug?

Currently we are not collecting metrics that are not enabled for top n queries, which causes certain measurements fields missing in local index sometimes. This is not a consistent behavior, idealy, when user configure "enabling/disabling top n queries by cpu", we should really just adding/removing the logic to push the records into the cpu metric heap, but whether the the top n queries are configured or not, we should always collect the metric.

How can one reproduce the bug?

  • Enable top n queries by latency only
  • Enable local index exporter
  • Wait untail the exporter exports to local index
  • Enable top n queries by memory or cpu
  • try to get historical top n queries with from and to

What is the expected behavior?

No error should be thrown and we should always collect the metrics

What is your host/environment?

Operating system, version.

Do you have any screenshots?

If applicable, add screenshots to help explain your problem.

Do you have any additional context?

Add any other context about the problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant