Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix that only aggregated metrics for filters which actually matched >0 things are reported #2050

Conversation

thjaeckle
Copy link
Member

  • also "forbid" to have multiple filters matching at the same time

@thjaeckle thjaeckle added the bug label Oct 24, 2024
@thjaeckle thjaeckle added this to the 3.6.2 milestone Oct 24, 2024
@thjaeckle thjaeckle requested a review from alstanchev October 24, 2024 12:33
@thjaeckle thjaeckle self-assigned this Oct 24, 2024
@thjaeckle thjaeckle force-pushed the bugfix/aggregation-metrics-with-non-zero branch from 0222a0d to 2d514dd Compare October 24, 2024 12:34
… 0 things are reported

* also "forbid" to have multiple filters matching at the same time
@thjaeckle thjaeckle force-pushed the bugfix/aggregation-metrics-with-non-zero branch from 2d514dd to a468d98 Compare October 24, 2024 12:35
Copy link
Contributor

@alstanchev alstanchev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@thjaeckle thjaeckle merged commit 275219c into eclipse-ditto:master Oct 25, 2024
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants