Skip to content

chore: unify codestyle in governor/plan.rs

Yaroslav Dynnikovrequested to merge
rosik/misc into master

Summary

  • chore: unify codestyle in governor/plan.rs

Here's the order of plan stages for memories:

$ grep "return Ok" src/governor/plan.rs
return Ok(ConfChange { conf_change }.into());
return Ok(TransferLeadership { to: new_leader }.into());
return Ok(Downgrade { req }.into());
return Ok(CreateReplicaset { master_id, replicaset_id, rpc, op }.into());
return Ok(UpdateTargetReplicasetMaster { op }.into());
return Ok(Replication { targets, master_id, replicaset_peers, req }.into());
return Ok(UpdateCurrentReplicasetMaster { old_master_id, demote, new_master_id, sync_and_promote, replicaset_id, op }.into());
return Ok(ProposeReplicasetStateChanges { op }.into());
return Ok(UpdateTargetVshardConfig { dml }.into());
return Ok(UpdateCurrentVshardConfig { targets, rpc, dml }.into());
return Ok(ShardingBoot { target, rpc, op }.into());
return Ok(ToOnline { target, rpc, req }.into());
return Ok(ApplySchemaChange { rpc, targets }.into());
  • Target branch: master
  • Changes should be cherry-picked to 24.1: no
  • Docs follow-up: not necessary

Merge request reports