Skip to content

Commit

Permalink
vttablet refactor: address review comments
Browse files Browse the repository at this point in the history
Signed-off-by: Sugu Sougoumarane <ssougou@gmail.com>
  • Loading branch information
sougou committed Jul 23, 2020
1 parent 1555def commit dce6346
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion config/tablet/default.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,6 @@ hotRowProtection:
maxConcurrency: 5 # hot_row_protection_concurrent_transactions

consolidator: enable|disable|notOnMaster # enable-consolidator, enable-consolidator-replicas
transitionGracePeriodSeconds: 0 # serving_state_grace_period
passthroughDML: false # queryserver-config-passthrough-dmls
streamBufferSize: 32768 # queryserver-config-stream-buffer-size
queryCacheSize: 5000 # queryserver-config-query-cache-size
Expand Down
2 changes: 1 addition & 1 deletion go/test/endtoend/cluster/mysqlctl_process.go
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ func (mysqlctl *MysqlctlProcess) StartProcess() (*exec.Cmd, error) {
// Stop executes mysqlctl command to stop mysql instance
func (mysqlctl *MysqlctlProcess) Stop() (err error) {
log.Infof("Shutting down MySQL: %d", mysqlctl.TabletUID)
defer log.Infof("MySQL shurdown complete: %d", mysqlctl.TabletUID)
defer log.Infof("MySQL shutdown complete: %d", mysqlctl.TabletUID)
tmpProcess, err := mysqlctl.StopProcess()
if err != nil {
return err
Expand Down
2 changes: 1 addition & 1 deletion go/test/endtoend/tabletmanager/tablet_health_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ func TestHealthCheck(t *testing.T) {

// stop replication, make sure we don't go unhealthy.
// TODO: replace with StopReplication once StopSlave has been removed
err = clusterInstance.VtctlclientProcess.ExecuteCommand("StopReplication", rTablet.Alias)
err = clusterInstance.VtctlclientProcess.ExecuteCommand("StopSlave", rTablet.Alias)
require.NoError(t, err)
err = clusterInstance.VtctlclientProcess.ExecuteCommand("RunHealthCheck", rTablet.Alias)
require.NoError(t, err)
Expand Down

0 comments on commit dce6346

Please sign in to comment.