Skip to content

Replace slave with replica in vtctl commands and flags#6428

Merged
sougou merged 7 commits intomasterfrom
rename-slave-replica
Jul 16, 2020
Merged

Replace slave with replica in vtctl commands and flags#6428
sougou merged 7 commits intomasterfrom
rename-slave-replica

Conversation

@deepthi
Copy link
Copy Markdown
Collaborator

@deepthi deepthi commented Jul 8, 2020

  • Rename StopSlave -> StopReplication. Deprecate StopSlave
  • Rename StartSlave -> StartReplication. Deprecate StartSlave.
  • Rename ChangeSlaveType -> ChangeTabletType. Deprecate ChangeSlaveType.
  • Rename flag wait_slave_timeout to wait_replicas_timeout for reparenting commands (ISM, PRS, ERS).
  • Rename flag wait_slave_timeout to wait_replicas_timeout for schema commands (ApplySchema, CopySchemaShard).
  • Rename flag schema_change_slave_timeout to schema_change_replicas_timeout.

All changes to be backwards-compatible. The old flags and commands are deprecated but not removed. They will be removed before the 8.0 release.

Signed-off-by: deepthi deepthi@planetscale.com

…cument old flags. ensure test coverage for old and new flags

Signed-off-by: deepthi <deepthi@planetscale.com>
deepthi and others added 2 commits July 8, 2020 17:07
Signed-off-by: deepthi <deepthi@planetscale.com>
…ication,StopReplication,ChangeTabletType}

Signed-off-by: Andrew Mason <amason@slack-corp.com>
Andrew Mason and others added 3 commits July 9, 2020 10:38
Signed-off-by: Andrew Mason <amason@slack-corp.com>
Signed-off-by: deepthi <deepthi@planetscale.com>
…e to old commands

Signed-off-by: deepthi <deepthi@planetscale.com>
@deepthi deepthi marked this pull request as ready for review July 14, 2020 02:53
@deepthi deepthi requested a review from sougou as a code owner July 14, 2020 02:53
@deepthi deepthi requested review from ajm188 and zmagg July 14, 2020 04:12
Signed-off-by: deepthi <deepthi@planetscale.com>
@sougou sougou merged commit 1d518f2 into master Jul 16, 2020
@shlomi-noach shlomi-noach deleted the rename-slave-replica branch July 16, 2020 06:29
@deepthi deepthi added this to the v7.0 milestone Jul 17, 2020
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.

3 participants