Skip to content

Commit 3dca6b1

Browse files
committed
Only treat CHAR(n) columns as Guids. Fixes #1565
This reverts a change in #1546 that was introduced to fix #1528. However, none of the integration tests fail locally when reverting this change, so it's not clear why it was necessary. Signed-off-by: Bradley Grainger <[email protected]>
1 parent 637d5c3 commit 3dca6b1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/MySqlConnector/ColumnReaders/ColumnReader.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,13 @@ public static ColumnReader Create(bool isBinary, ColumnDefinitionPayload columnD
4545
return BitColumnReader.Instance;
4646

4747
case ColumnType.String:
48-
case ColumnType.VarString:
4948
if (connection.GuidFormat == MySqlGuidFormat.Char36 && columnDefinition.ColumnLength / ProtocolUtility.GetBytesPerCharacter(columnDefinition.CharacterSet) == 36)
5049
return GuidChar36ColumnReader.Instance;
5150
if (connection.GuidFormat == MySqlGuidFormat.Char32 && columnDefinition.ColumnLength / ProtocolUtility.GetBytesPerCharacter(columnDefinition.CharacterSet) == 32)
5251
return GuidChar32ColumnReader.Instance;
53-
goto case ColumnType.VarChar;
52+
goto case ColumnType.VarString;
5453

54+
case ColumnType.VarString:
5555
case ColumnType.VarChar:
5656
case ColumnType.TinyBlob:
5757
case ColumnType.Blob:

0 commit comments

Comments
 (0)