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

Update HealthCheck.pm #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update HealthCheck.pm #138

wants to merge 1 commit into from

Conversation

DaoWind
Copy link

@DaoWind DaoWind commented Aug 14, 2020

I want to know why use error_count to forbid re-lock monitor when connection loss?
I set pass 0 to connect event if connection loss, so it will relock again.
Is it OK for this change?

I want to know why use error_count to forbid re-lock monitor when connection loss?
I set pass 0 to connect event if connection loss, so it will relock again. 
Is it OK for this change?
@DaoWind
Copy link
Author

DaoWind commented Aug 17, 2020

I just found this issue, and want to confirm this issue(after connection loss, it connect agian, but the lock was released).
I think it is not better for another MHA manager can start(even it not too much harmful for whole cluster.)

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.

1 participant