Update for new errcheck violations after rebase

Signed-off-by: Andrew Mason <amason@slack-corp.com>

.golangci.yml

@@ -109,6 +109,9 @@ issues:
     - path: '^go/vt/vtgate/.*_test.go'
       linters:
       - errcheck
+    - path: '^go/vt/vtgr/'
+      linters:
+      - errcheck
     - path: '^go/vt/vttablet/(customrule|filelogger|grpctmserver|onlineddl|sandboxconn|tabletserver)/'
       linters:
       - errcheck

go/vt/vtgate/planbuilder/gen4_planner.go

@@ -109,7 +109,7 @@ func newBuildSelectPlan(sel *sqlparser.Select, reservedVars *sqlparser.ReservedV
 
 	directives := sqlparser.ExtractCommentDirectives(sel.Comments)
 	if directives.IsSet(sqlparser.DirectiveScatterErrorsAsWarnings) {
-		visit(plan, func(logicalPlan logicalPlan) (bool, logicalPlan, error) {
+		_, _ = visit(plan, func(logicalPlan logicalPlan) (bool, logicalPlan, error) {
 			switch plan := logicalPlan.(type) {
 			case *route:
 				plan.eroute.ScatterErrorsAsWarnings = true

go/vt/vtgate/vstream_manager.go

@@ -451,7 +451,7 @@ func (vs *vstream) streamFromTablet(ctx context.Context, sgtid *binlogdatapb.Sha
 
 		errCh := make(chan error, 1)
 		go func() {
-			tabletConn.StreamHealth(ctx, func(shr *querypb.StreamHealthResponse) error {
+			_ = tabletConn.StreamHealth(ctx, func(shr *querypb.StreamHealthResponse) error {
 				var err error
 				if ctx.Err() != nil {
 					err = fmt.Errorf("context has ended")