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

Use GLOBAL to prevent mysql deadlock #336

Conversation

codytaylor
Copy link
Contributor

@SuperQ
In mysql 5.7.22 we encountered an issue where the entire process would completely lock up. We were able to solve it by modifying this one file and also removing the '--collect.binlog_size' param from the mysqld exporter.

Here is the bug report: https://bugs.mysql.com/bug.php?id=92108

@SuperQ
Copy link
Member

SuperQ commented Sep 24, 2018

Very interesting, seems reasonable to me.

@SuperQ
Copy link
Member

SuperQ commented Sep 24, 2018

Please sign your commit with git commit -s --amend.

@SuperQ
Copy link
Member

SuperQ commented Sep 24, 2018

Hrm, it looks like your commit history is a little strange now.

@codytaylor
Copy link
Contributor Author

Yeah, trying out a new tool called sublime merge. Looks like it did a pull/merge before I pushed the signed commit. No further changes though.

Need me to figure out how to remove that?

Signed-off-by: Cody Taylor <cody@codytaylor.org>
@SuperQ
Copy link
Member

SuperQ commented Sep 24, 2018

Yes, it would be nice. It seems like the commit history is making the DCO bot confused.

@codytaylor codytaylor force-pushed the use-GLOBAL-to-prevent-complete-mysql-lockup branch from 04d7ccb to 6bf0ec9 Compare September 24, 2018 20:01
@codytaylor
Copy link
Contributor Author

Looks good now

Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

LGTM

@SuperQ SuperQ merged commit ae7180b into prometheus:master Sep 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants