-
Notifications
You must be signed in to change notification settings - Fork 12
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
PMM-12650 vacuum monitoring #166
base: main
Are you sure you want to change the base?
Conversation
Changes to be committed: modified: cmd/postgres_exporter/postgres_exporter.go modified: cmd/postgres_exporter/queries.go Adding vacuum monitoring and adjusting pg_stat_activity for better troubleshooting.
@@ -142,12 +221,13 @@ var queryOverrides = map[string][]OverrideQuery{ | |||
( | |||
SELECT | |||
datname, | |||
pid, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as far as I remember this pid value creates high cardinality and brings a problem with bad performance and high disk space usage.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BupycHuk Are we going to try it and if needed discart it based on QAs results? Or discart it before merge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's discard it before merge, because we've faced this problem already in past
@heniek Could you discard changes related to PID? Thank you. |
Hey, |
Changes to be committed:
modified: cmd/postgres_exporter/postgres_exporter.go
modified: cmd/postgres_exporter/queries.go
PMM-12650 Adding vacuum monitoring and adjusting pg_stat_activity for better troubleshooting.