Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

Reduce non-primary rollup log spew #1974

Merged
merged 1 commit into from
Apr 19, 2021
Merged

Reduce non-primary rollup log spew #1974

merged 1 commit into from
Apr 19, 2021

Conversation

shanson7
Copy link
Collaborator

Looks like this log line is in the wrong for loop, so it ends up spewing for requests with many series with cnt going up by one each time.

@shanson7 shanson7 requested a review from Dieterbe April 15, 2021 11:12
@Dieterbe
Copy link
Contributor

oops. thanks @shanson7 !
@fkaleo FYI not sure if you were tracking this log line. but without this fix we would spew way too many of these lines.

@Dieterbe Dieterbe merged commit cd09d04 into grafana:master Apr 19, 2021
@fkaleo
Copy link
Contributor

fkaleo commented Apr 19, 2021

oops. thanks @shanson7 !
@fkaleo FYI not sure if you were tracking this log line. but without this fix we would spew way too many of these lines.

cannot believe I missed it, thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Dieterbe Dieterbe Dieterbe approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@shanson7 @Dieterbe @fkaleo