hutiefang76 commented on PR #12543: URL: https://github.com/apache/seatunnel/pull/12543#issuecomment-5876522313
Closing this for now after checking the adjacent timestamp and unsigned query types. On JDBC 1.3.1.0, `TIMESTAMP_S`, `TIMESTAMP_MS`, and `TIMESTAMP_NS` all report JDBC `TIMESTAMP`. The existing query path returns `TIMESTAMP`/`LocalDateTime`, but this mapper sends their native names to the converter's STRING fallback, changing the schema and the actual row values to strings. Also, this PR on its own narrows `UTINYINT` query output from SMALLINT to TINYINT; it needs the unsigned converter fix in #12508. The UUID/JSON/DECIMAL tests covered the reported failures, but missed these regressions in previously working queries. I will keep this withdrawn until the existing mappings are preserved and the dependency is resolved. Thanks for reviewing. -- 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]
