Skip to content

Commit

Permalink
Remove unnecessary param
Browse files Browse the repository at this point in the history
Signed-off-by: Rohit Nayak <rohit@planetscale.com>
  • Loading branch information
rohit-nayak-ps committed Jan 29, 2025
1 parent ff0279f commit 853bb6f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion go/vt/vttablet/tabletmanager/vdiff/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ type controller struct {
TableDiffPhaseTimings *stats.Timings
}

func newController(ctx context.Context, row sqltypes.RowNamedValues, dbClientFactory func() binlogplayer.DBClient,
func newController(row sqltypes.RowNamedValues, dbClientFactory func() binlogplayer.DBClient,
ts *topo.Server, vde *Engine, options *tabletmanagerdata.VDiffOptions) (*controller, error) {

log.Infof("VDiff controller initializing for %+v", row)
Expand Down
2 changes: 1 addition & 1 deletion go/vt/vttablet/tabletmanager/vdiff/engine.go
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ func (vde *Engine) retry(ctx context.Context, err error) {
// addController creates a new controller using the given vdiff record and adds it to the engine.
// You must already have the main engine mutex (mu) locked before calling this.
func (vde *Engine) addController(row sqltypes.RowNamedValues, options *tabletmanagerdata.VDiffOptions) error {
ct, err := newController(vde.ctx, row, vde.dbClientFactoryDba, vde.ts, vde, options)
ct, err := newController(row, vde.dbClientFactoryDba, vde.ts, vde, options)
if err != nil {
return fmt.Errorf("controller could not be initialized for stream %+v on tablet %v",
row, vde.thisTablet.Alias)
Expand Down
2 changes: 1 addition & 1 deletion go/vt/vttablet/tabletmanager/vdiff/framework_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -693,7 +693,7 @@ func (tvde *testVDiffEnv) createController(t *testing.T, id int) *controller {

func (tvde *testVDiffEnv) newController(t *testing.T, controllerQR *sqltypes.Result) *controller {
ctx := context.Background()
ct, err := newController(ctx, controllerQR.Named().Row(), tvde.dbClientFactory, tstenv.TopoServ, tvde.vde, tvde.opts)
ct, err := newController(controllerQR.Named().Row(), tvde.dbClientFactory, tstenv.TopoServ, tvde.vde, tvde.opts)
require.NoError(t, err)
ctx2, cancel := context.WithCancel(ctx)
ct.cancel = cancel
Expand Down

0 comments on commit 853bb6f

Please sign in to comment.