DanielLeens commented on PR #11069: URL: https://github.com/apache/seatunnel/pull/11069#issuecomment-5082450578
Correction to my previous comment: the shell stripped the inline code spans while I was posting it, so the technical references were rendered incomplete. The intended evidence is: - Current `dev` already covers PostgreSQL CDC `TIMESTAMP_TZ` through the later merged #10724. - The current `dev` implementation wires `TIMESTAMP_TZ` to `convertToTimestampTz(serverTimeZone)`. - The current tests cover ISO offsets, IANA zone ids, space-separated datetime strings, short hour-only offsets, UTC literals, and invalid-format handling. So the conclusion is unchanged: this PR was a useful contribution, but the current branch has a newer and broader implementation now. Because this PR conflicts with that newer implementation, merging it would not be the right rescue path anymore. -- 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]
