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

Add metrics for MergeTree parts (Wide/Compact/InMemory) types #19381

Merged
merged 4 commits into from
Jan 22, 2021

Conversation

azat
Copy link
Collaborator

@azat azat commented Jan 21, 2021

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Add metrics for MergeTree parts (Wide/Compact/InMemory) types

Follow up for #19122

- PartsWide
- PartsCompact
- PartsInMemory
…01600_parts_states_metrics

Without this setting the test will pass only when the table will be
actually removed by the background worker of the Atomic engine.

In CI this is not required since
database_atomic_wait_for_drop_and_detach_synchronously is set explicitly
in via tests/users.d.
@robot-clickhouse robot-clickhouse added the pr-improvement Pull request with some product improvements label Jan 21, 2021
Copy link
Member

@alexey-milovidov alexey-milovidov left a comment

Choose a reason for hiding this comment

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

LGTM.

I thought about different implementation - just adding MetricIncrement member to Part structs.

@alexey-milovidov alexey-milovidov self-assigned this Jan 21, 2021
@azat
Copy link
Collaborator Author

azat commented Jan 22, 2021

I thought about different implementation - just adding MetricIncrement member to Part structs.

Yes this is an option, and it will look simpler, but will require additional MergeTreeDataPartType::UNKNOWN handling (or __builtin_unreachable for it, which looks hackish)

@alexey-milovidov alexey-milovidov merged commit 9818bd3 into ClickHouse:master Jan 22, 2021
@azat azat deleted the parts-types-metrics branch January 22, 2021 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-improvement Pull request with some product improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants