Properly wrap an error to avoid losing stack trace in prints

zachmu · zachmu · commit eff2c25a3314 · 2025-01-28T17:53:37.000-08:00
diff --git a/sql/planbuilder/ddl.go b/sql/planbuilder/ddl.go
@@ -1650,7 +1650,7 @@ func (b *Builder) resolveColumnDefaultExpression(inScope *scope, columnDef *sql.
 
 	parsed, err := b.parser.ParseSimple(fmt.Sprintf("SELECT %s", def))
 	if err != nil {
-		err := fmt.Errorf("%w: %s", sql.ErrInvalidColumnDefaultValue.New(def), err)
+		err := sql.ErrInvalidColumnDefaultValue.Wrap(err, def)
 		b.handleErr(err)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1650,7 +1650,7 @@ func (b Builder) resolveColumnDefaultExpression(inScope scope, columnDef *sql.`
`1650`	`1650`
`1651`	`1651`	`parsed, err := b.parser.ParseSimple(fmt.Sprintf("SELECT %s", def))`
`1652`	`1652`	`if err != nil {`
`1653`		`- err := fmt.Errorf("%w: %s", sql.ErrInvalidColumnDefaultValue.New(def), err)`
	`1653`	`+ err := sql.ErrInvalidColumnDefaultValue.Wrap(err, def)`
`1654`	`1654`	`b.handleErr(err)`
`1655`	`1655`	`}`
`1656`	`1656`