MySQL client should provide columnDescriptors with LONGVARCHAR JDBCType for TEXT types

[DavideD]

See this issue in Quarkus.

Is there any particular reason for this or it's a bug?

@fedinskiy:

I took a look into code for Postgress[1] and MySQL[2] and it looks like the text case is not handled properly in the latter.

[1] https://github.com/eclipse-vertx/vertx-sql-client/blob/master/vertx-pg-client/src/main/java/io/vertx/pgclient/impl/codec/DataType.java
[2] https://github.com/eclipse-vertx/vertx-sql-client/blob/master/vertx-mysql-client/src/main/java/io/vertx/mysqlclient/impl/datatype/DataType.java

[vietj]

this requires investigations @DavideD , @BillyYccc any idea ?

[BillyYccc]

I ran an HR reproducer and see what Vert.x mysql client returns in the row is a java.lang.String. Then I checked the HR code in
https://github.com/hibernate/hibernate-reactive/blob/16fe525025c7d47a20857c7d38033da296531d74/hibernate-reactive-core/src/main/java/org/hibernate/reactive/adaptor/impl/ResultSetAdaptor.java#L715-L731 and https://github.com/hibernate/hibernate-reactive/blob/16fe525025c7d47a20857c7d38033da296531d74/hibernate-reactive-core/src/main/java/org/hibernate/reactive/adaptor/impl/ResultSetAdaptor.java#L372.

It seems the HR will depend on the Vert.x io.vertx.sqlclient.desc.ColumnDescriptor#jdbcType to determine what Java type will be returned, and the mysql driver did return a JDBCType.BLOB for both MySQL BLOB and TEXT types.

I think we could add some io.vertx.mysqlclient.impl.datatype#DataType enum values to distinguish between BLOB and TEXT type description(the types are distinguished by the column collation id - see #482).

[gavinking]

the mysql driver did return a JDBCType.BLOB for both MySQL BLOB and TEXT types.

But surely that's simply wrong! TEXT is MySQL's CLOB type, it's not a BLOB.