panbingkun commented on PR #46655: URL: https://github.com/apache/spark/pull/46655#issuecomment-2126901464
> > IMO, it seems that the 3.x version is more reasonable, > > It's unreasonable to me. The 2.x and MySQL official both use `java.sql.Types.SMALLINT + Signed signal` = `[UN]SIGNED SMALLINT` Okay, I see what you mean. What you say is `reasonable` is that you focus on the value returned in `3.x` and do not make full use of `isSigned`. What I say is `reasonable` is that you focus on its own division of `UnsignedSmallIntColumn` and `SignedSmallIntColumn` without paying attention to its `isSigned`. Let me investigate again. it should have been like `this` in the original version of `3.x` (from 3.0.3) https://github.com/mariadb-corporation/mariadb-connector-j/commit/22cff70de2be679e925a0d231123ad72f30ffdd6 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
