DanielLeens commented on PR #10724: URL: https://github.com/apache/seatunnel/pull/10724#issuecomment-4410996405
Hi @doyong365, thanks for the ping. I checked the current PR head again. The Build is green now, but there still has not been a new commit after my latest source-level review on May 4, 2026. So I am not doing a new full re-review yet in this round. My blocker from that latest review is still the same one: - the normal Xugu sink write path still converts `OffsetDateTime` to `LocalDateTime` before `PreparedStatement.setTimestamp(...)`, so `TIMESTAMP_TZ` can still lose the offset on write. Once there is a new commit that either preserves that value correctly, or explicitly fails/documents the downgrade instead of writing it silently, I am happy to re-review quickly. -- 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]
