sunxiaojian commented on code in PR #8132:
URL: https://github.com/apache/seatunnel/pull/8132#discussion_r1857850330
##########
seatunnel-api/src/main/java/org/apache/seatunnel/api/table/converter/BasicTypeDefine.java:
##########
@@ -31,6 +31,7 @@ public class BasicTypeDefine<T> implements Serializable {
protected String columnType;
// e.g. `varchar` for MySQL
protected String dataType;
+ protected int sqlType;
Review Comment:
> I think we can use nativeType instead of sqlType.
no, like 'INT' cannot be converted to standard JDBCType, SQLType doesn't
need to worry about this issue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
