davidzollo commented on PR #10724: URL: https://github.com/apache/seatunnel/pull/10724#issuecomment-4670920801
Thanks for the follow-up here. I see a few issues: 1. Spark `TIMESTAMP_TZ` serialization still looks unsafe for negative offsets. The current decimal encoding for `OffsetDateTime` appears to produce an invalid `BigDecimal` for values like `-08:00`, so this path can fail once more JDBC timestamp columns start flowing through Spark. 2. The new non-UTC MySQL E2E test only checks the field type, not the actual instant/value, so it would not catch the timezone-shift regression this PR is trying to prevent. -- 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]
