diff --git a/go/vt/vtgate/executor_select_test.go b/go/vt/vtgate/executor_select_test.go index e83090f1eec..997757330df 100644 --- a/go/vt/vtgate/executor_select_test.go +++ b/go/vt/vtgate/executor_select_test.go @@ -596,7 +596,7 @@ func TestSelectKeyRangeUnique(t *testing.T) { func TestSelectIN(t *testing.T) { executor, sbc1, sbc2, sbclookup := createExecutorEnv() - // Constant in IN clause, not a bind variable. + // Constant in IN clause is just a number, not a bind variable. _, err := executorExec(executor, "select id from user where id in (1)", nil) if err != nil { t.Error(err) @@ -614,7 +614,7 @@ func TestSelectIN(t *testing.T) { t.Errorf("sbc2.Queries: %+v, want nil\n", sbc2.Queries) } - // Constant IN is just a couple numbers, not bind variables. + // Constants in IN clause are just numbers, not bind variables. // They result in two different queries on two shards. sbc1.Queries = nil sbc2.Queries = nil diff --git a/go/vt/vtgate/executor_test.go b/go/vt/vtgate/executor_test.go index 5e3349db10f..8af068afeed 100644 --- a/go/vt/vtgate/executor_test.go +++ b/go/vt/vtgate/executor_test.go @@ -1841,7 +1841,7 @@ func TestExecutorUnrecognized(t *testing.T) { func TestExecutorMessageAckSharded(t *testing.T) { executor, sbc1, sbc2, _ := createExecutorEnv() - // Constant in IN clause, not a bind variable. + // Constant in IN clause is just a number, not a bind variable. ids := []*querypb.Value{{ Type: sqltypes.VarChar, Value: []byte("1"), @@ -1860,7 +1860,7 @@ func TestExecutorMessageAckSharded(t *testing.T) { t.Errorf("sbc2.MessageIDs: %+v, want nil\n", sbc2.MessageIDs) } - // Constant IN is just a couple numbers, not bind variables. + // Constants in IN clause are just numbers, not bind variables. // They result in two different MessageIDs on two shards. sbc1.MessageIDs = nil sbc2.MessageIDs = nil