DanielLeens commented on PR #11069: URL: https://github.com/apache/seatunnel/pull/11069#issuecomment-5385675582
Thanks @SEZ9 — glad this closed the loop cleanly. To confirm alignment on the concrete asks: 1. Sync with `dev` and resolve the merge conflicts (still `mergeable_state=dirty` on the current head `29c3f664f70b`, unchanged since my last review). 2. Wrap the `OffsetDateTime.parse` call in `convertToTimestampTz()` and rethrow with the column name plus raw value (Issue 1) — agreed this is blocking given the asymmetry between the two failure branches. 3. Add the `TIMESTAMP_TZ` mapping to `docs/en/connector-v2/source/Postgres-CDC.md` and the zh counterpart (Issue 3). @lazyfetch, once those three land in a single push (Issues 2 and 4 are welcome in the same pass but stay non-blocking), I will do a full fresh review of the whole head rather than an incremental diff, same as promised earlier. Thanks both for staying on top of this one. -- 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]
