Skip to content

Refactor SlaveLag throttler as ReplicaLag#144

Merged
Sinclert merged 3 commits into
masterfrom
sinclert/refactor-slave-lag-throttler
Aug 31, 2023
Merged

Refactor SlaveLag throttler as ReplicaLag#144
Sinclert merged 3 commits into
masterfrom
sinclert/refactor-slave-lag-throttler

Conversation

@Sinclert

Copy link
Copy Markdown

This PR refactors the legacy named SlaveLag throttler to nowadays terminology (ReplicaLag). To maintain compatibility with existing applications, this throttler can be selected using any of the following values:

  • replica_lag_throttler (new).
  • slave_lag_throttler (deprecated).

Notes

Given this gem still supports MySQL 5.7, there are certain MySQL commands which could not be refactored:

  • The SHOW SLAVE STATUS command (reference).
  • The START / STOP SLAVE commands (reference).
  • The GRANT REPLICATION SLAVE command (reference).
  • The --log-slave-updates config option (reference).

Comment thread lib/lhm/throttler/replica_lag.rb
Comment thread CHANGELOG.md Outdated
Comment thread spec/integration/integration_helper.rb
@Sinclert Sinclert force-pushed the sinclert/refactor-slave-lag-throttler branch from 6c1b7a7 to b59177a Compare August 30, 2023 12:40
@Sinclert Sinclert requested a review from driv3r August 30, 2023 13:56

@driv3r driv3r left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

@Sinclert Sinclert force-pushed the sinclert/refactor-slave-lag-throttler branch from b59177a to 864d7b9 Compare August 31, 2023 13:07
@Sinclert Sinclert merged commit bfc1f7e into master Aug 31, 2023
@Sinclert Sinclert deleted the sinclert/refactor-slave-lag-throttler branch August 31, 2023 13:10
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