Skip to content

Fix metric Query#34224

Merged
tavplubix merged 1 commit intoClickHouse:masterfrom
CurtizJ:fix-query-metric
Feb 2, 2022
Merged

Fix metric Query#34224
tavplubix merged 1 commit intoClickHouse:masterfrom
CurtizJ:fix-query-metric

Conversation

@CurtizJ
Copy link
Copy Markdown
Member

@CurtizJ CurtizJ commented Feb 1, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehaviour in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix metric Query, which shows number of executing queries. In last several releases it was always 0.

Fixes #34014.
Increment of this metric was accidentally removed in #31636.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Feb 1, 2022
@tavplubix tavplubix self-assigned this Feb 1, 2022
@tavplubix tavplubix added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Feb 1, 2022
@tavplubix
Copy link
Copy Markdown
Member

ClickHouse special build check (actions) - unrelated, was broken in master

@tavplubix tavplubix merged commit 7304fd6 into ClickHouse:master Feb 2, 2022
robot-clickhouse pushed a commit that referenced this pull request Feb 2, 2022
robot-clickhouse pushed a commit that referenced this pull request Feb 3, 2022
CurtizJ added a commit that referenced this pull request Feb 14, 2022
CurtizJ added a commit that referenced this pull request Feb 14, 2022
@Felixoid Felixoid added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default pr-must-backport Pull request should be backported intentionally. Use this label with great care!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

system.metrics name='Query' and system.metric_log CurrentMetric_Query gauge doesn't calculate anymore

4 participants