Skip to content

Commit 841f067

Browse files
committed
Bug fixes
1 parent 97605e7 commit 841f067

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

sql/types/conversion.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -721,7 +721,7 @@ func GeneralizeTypes(a, b sql.Type) sql.Type {
721721
return generalizeNumberTypes(a, b)
722722
}
723723

724-
if IsText(a) || IsText(b) {
724+
if IsText(a) && IsText(b) {
725725
return a
726726
}
727727

sql/types/conversion_test.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,15 +198,17 @@ func TestGeneralizeTypes(t *testing.T) {
198198
{Int8, Int8, Int8},
199199
{Boolean, Int64, Int64},
200200
{Boolean, Boolean, Boolean},
201-
{Text, Text, LongText},
201+
{Text, Text, Text},
202+
{Text, LongText, Text},
202203
{Text, Float64, LongText},
203204
{Int64, Text, LongText},
204205
{Int8, Null, Int8},
205206
{Time, Time, Time},
206207
{Time, Date, DatetimeMaxPrecision},
207208
{Date, Date, Date},
208209
{Date, Timestamp, DatetimeMaxPrecision},
209-
{Timestamp, Timestamp, TimestampMaxPrecision},
210+
{Timestamp, Timestamp, Timestamp},
211+
{Timestamp, TimestampMaxPrecision, TimestampMaxPrecision},
210212
{Timestamp, Datetime, DatetimeMaxPrecision},
211213
{Null, Int64, Int64},
212214
{Null, Null, Null},

0 commit comments

Comments
 (0)