FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

pkg/parcacol: Sum into a single series if no sum by given by metalmatze · Pull Request #5580 · parca-dev/parca · GitHub

pkg/parcacol: Sum into a single series if no sum by given - #5580

Merged
metalmatze merged 1 commit into
mainfrom
sum-by-empty-single-series
Apr 7, 2025
Merged

pkg/parcacol: Sum into a single series if no sum by given#5580
metalmatze merged 1 commit into
mainfrom
sum-by-empty-single-series

Conversation

Copy link
Copy Markdown
Member

Previously we would sum by all labels but this doesn't scale when there are hundreds of series available. Instead, we now have sum by() create a single series, much like in Prometheus. This is easier to draw and less confusing to the users.

Previously we would sum by all labels but this doesn't scale when there are hundreds of series available. Instead, we now have `sum by()` create a single series, much like in Prometheus. This is easier to draw and less confusing to the users.

alwaysmeticulous Bot commented Apr 7, 2025
edited
Loading

Copy link
Copy Markdown

🤖 Meticulous evaluated 50 user flows and took 315 visual snapshots. Meticulous has not yet run on cc3456b of the main branch and so there was nothing to compare against.

If you recently setup Meticulous, this is expected. Meticulous will start reporting comparisons for new pull requests after the next commit to the main branch.

Last updated for commit f608093. This comment will update as new commits are pushed.

metalmatze merged commit 2e7aa0f into main Apr 7, 2025
metalmatze deleted the sum-by-empty-single-series branch April 7, 2025 15:43
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL