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

Introduced new HG attribute 'monitor_slave_lag_when_null' which takes precedence over 'mysql_thread_monitor_slave_lag_when_null' #4528

Merged
merged 6 commits into from
May 20, 2024

Conversation

rahim-kanji
Copy link
Collaborator

@rahim-kanji rahim-kanji commented Apr 25, 2024

Introduced a new hostgroup attribute 'monitor_slave_lag_when_null' in 'mysql_hostgroup_attibutes->hostgroup_settings', which takes precedence over 'mysql_thread_monitor_slave_lag_when_null' if both are configured.

Example usage:
INSERT INTO mysql_hostgroup_attributes (hostgroup_id, hostgroup_settings) VALUES (1, '{"monitor_slave_lag_when_null": 100}');

Note:
* Permitted values for 'mysql-monitor_slave_lag_when_null' also apply to the 'monitor_slave_lag_when_null' attribute.
* If 'Seconds_Behind_Master' is negative, it will bring previously shunned server, due to replication lag, back online.
* True 'Seconds_Behind_Master' values returned by the server will be logged in 'mysql_server_replication_lag_log table. (Previously negative values were logged as null)

Closes #4521

…p settings, which takes precedence over 'mysql_thread_monitor_slave_lag_when_null' if both are configured.
@rahim-kanji rahim-kanji marked this pull request as ready for review April 26, 2024 07:29
@rahim-kanji rahim-kanji force-pushed the v2.x_monitor_slave_lag_when_null_4521 branch from bc2d29d to 3ef9c5e Compare May 3, 2024 12:23
@rahim-kanji
Copy link
Collaborator Author

retest this please

1 similar comment
@mirostauder
Copy link
Collaborator

retest this please

@JavierJF
Copy link
Collaborator

Retest this please.

@renecannao
Copy link
Contributor

retest this please

These messages were introduced by #44678343, and got deprecated at the same
time that 'admin-checksum_*' were deprecated (#d61d9892). Their original
motivation is summarized in the following note:

```
LOGGING-NOTE: Log only if the remote checksum has just been updated. Avoids
constant logging when 'global_checksum' are not used to prevent individual
modules fetching.
```

These two messages mistakenly survived this deprecation with the introduction
of table 'mysql_servers_v2'.
@JavierJF
Copy link
Collaborator

Retest this please.

1 similar comment
@JavierJF
Copy link
Collaborator

Retest this please.

@renecannao renecannao merged commit cdfcfdc into v2.x May 20, 2024
43 of 45 checks passed
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.

Implement setting in mysql_hostgroup_attributes to override global mysql-monitor_slave_lag_when_null
4 participants