Review suggestion

frouioui · frouioui · commit a1071935f2e0 · 2024-09-30T02:19:29.000-06:00
Signed-off-by: Florent Poinsard &lt;florent.poinsard@outlook.fr&gt;
diff --git a/go/vt/mysqlctl/builtinbackupengine.go b/go/vt/mysqlctl/builtinbackupengine.go
@@ -1055,7 +1055,7 @@ func (be *BuiltinBackupEngine) restoreFiles(ctx context.Context, params RestoreP
 			// We check for errors before checking if the context is canceled on purpose, if there was an
 			// error, the context would have been canceled already.
 			if rec.HasErrors() {
-				params.Logger.Infof("Failed to restore files due to error.")
+				params.Logger.Errorf("Failed to restore files due to error: %v", bh.Error())
 				return
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -1055,7 +1055,7 @@ func (be *BuiltinBackupEngine) restoreFiles(ctx context.Context, params RestoreP`
`1055`	`1055`	`// We check for errors before checking if the context is canceled on purpose, if there was an`
`1056`	`1056`	`// error, the context would have been canceled already.`
`1057`	`1057`	`if rec.HasErrors() {`
`1058`		`- params.Logger.Infof("Failed to restore files due to error.")`
	`1058`	`+ params.Logger.Errorf("Failed to restore files due to error: %v", bh.Error())`
`1059`	`1059`	`return`
`1060`	`1060`	`}`
`1061`	`1061`