Check for errors

dolthub · Feb 28, 2025 · 0830e3b · 0830e3b
1 parent 27e95f0
commit 0830e3b
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/go/cmd/dolt/commands/engine/sql_print.go b/go/cmd/dolt/commands/engine/sql_print.go
@@ -83,6 +83,9 @@ func prettyPrintResultsWithSummary(ctx *sql.Context, resultFormat PrintResultFor
 	var err error
 	var numRows int
 
+	// Function to print results. A function is required because we need to wrap the whole process in a swap of
+	// IO streams. This is done with cli.ExecuteWithStdioRestored, which requires a resultless function. As
+	// a result, we need to depend on side effects to numRows and err to determine if it was successful.
 	printEm := func() {
 		writerStream := cli.CliOut
 		if pageResults {
@@ -126,6 +129,9 @@ func prettyPrintResultsWithSummary(ctx *sql.Context, resultFormat PrintResultFor
 	} else {
 		printEm()
 	}
+	if err != nil {
+		return err
+	}
 
 	// if there is no row data and result format is JSON, then create empty JSON.
 	if resultFormat == FormatJson && numRows == 0 {