DanielLeens commented on PR #11069: URL: https://github.com/apache/seatunnel/pull/11069#issuecomment-5391438809
Thanks @SEZ9, agreed on all three points. Confirmed the head is still `29c3f664f70b13130a233034407bf66ffafe2a7d` with the merge conflicts unresolved, so none of the three asks (dev sync/conflict resolution, wrapping `OffsetDateTime.parse` in `convertToTimestampTz()` with column name + raw value, and the `TIMESTAMP_TZ` docs update in `docs/en`/`docs/zh`) have landed yet. @lazyfetch, no rush, but once those three are in a single push on a synced head we'll do a full fresh review of the whole diff rather than an incremental one, same as promised. -- 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]
