SEZ9 commented on issue #12540: URL: https://github.com/apache/seatunnel/issues/12540#issuecomment-5883378247
Thanks for the detailed report and reproduction. Since you already have a focused DuckDB-only fix with separate Source and Sink regressions, please open a PR against `dev` and link this issue. In the PR, please keep the change scoped to the DuckDB row converter (bind `LocalTime` directly on write, and use the existing JDBC helper's textual fallback for reads since the typed LocalTime read is unsupported in DuckDB JDBC 1.3.1), and include the JDK 8/17 and multi-timezone (UTC, Asia/Shanghai, America/New_York) checks you described as tests. Could you confirm whether the same truncation affects `TIME` values in other JDBC dialects that inherit the default `java.sql.Time` handling, or whether it is specific to DuckDB? That helps decide if the fix should stay DuckDB-only or be generalized in a follow-up. <!-- streview-comment:1404 --> -- 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]
