Check results instead of err when both are present

Signed-off-by: Rafer Hazen <[email protected]>
vitessio · Jul 25, 2024 · 5ef68ca · 5ef68ca
1 parent 8e9bccb
commit 5ef68ca
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/go/vt/vtgate/engine/fk_verify.go b/go/vt/vtgate/engine/fk_verify.go
@@ -82,7 +82,7 @@ func (f *FkVerify) TryExecute(ctx context.Context, vcursor VCursor, bindVars map
 	}
 
 	result, err := vcursor.ExecutePrimitive(ctx, f.Exec, bindVars, wantfields)
-	if err == nil {
+	if result != nil {
 		result.MergeStats(stats)
 	}
 	return result, err

diff --git a/go/vt/vtgate/engine/insert_select.go b/go/vt/vtgate/engine/insert_select.go
@@ -315,7 +315,7 @@ func (ins *InsertSelect) execInsertSharded(ctx context.Context, vcursor VCursor,
 	}
 
 	result, err := ins.insertIntoShardedTable(ctx, vcursor, bindVars, selectResult)
-	if err == nil {
+	if result != nil {
 		result.MergeStats(selectResult.stats)
 	}