Skip to content

Commit b8a66a6

Browse files
elianddbclaude
andcommitted
Enhance enum error messages with column names
- Added ErrDataTruncatedForColumn enhancement in INSERT execution (sql/rowexec/insert.go) - Added ErrDataTruncatedForColumn enhancement in UPDATE execution (sql/expression/set.go) - Error messages now show actual column names instead of empty strings - Improved MySQL compatibility for enum validation errors Before: 'Data truncated for column \'\'' After: 'Data truncated for column \'enum_col\'' 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 904472a commit b8a66a6

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

sql/expression/set.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,8 @@ func (s *SetField) Eval(ctx *sql.Context, row sql.Row) (interface{}, error) {
7777
// Fill in error with information
7878
if types.ErrLengthBeyondLimit.Is(err) {
7979
return nil, sql.NewWrappedTypeConversionError(val, getField.fieldIndex, types.ErrLengthBeyondLimit.New(val, getField.Name()))
80+
} else if types.ErrDataTruncatedForColumn.Is(err) {
81+
return nil, sql.NewWrappedTypeConversionError(val, getField.fieldIndex, types.ErrDataTruncatedForColumn.New(getField.Name()))
8082
}
8183
return nil, sql.NewWrappedTypeConversionError(val, getField.fieldIndex, err)
8284
}

sql/rowexec/insert.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,8 @@ func (i *insertIter) Next(ctx *sql.Context) (returnRow sql.Row, returnErr error)
140140
cErr = types.ErrLengthBeyondLimit.New(row[idx], col.Name)
141141
} else if sql.ErrNotMatchingSRID.Is(cErr) {
142142
cErr = sql.ErrNotMatchingSRIDWithColName.New(col.Name, cErr)
143+
} else if types.ErrDataTruncatedForColumn.Is(cErr) {
144+
cErr = types.ErrDataTruncatedForColumn.New(col.Name)
143145
}
144146
return nil, sql.NewWrappedInsertError(origRow, cErr)
145147
}

0 commit comments

Comments
 (0)