DanielLeens commented on PR #10472: URL: https://github.com/apache/seatunnel/pull/10472#issuecomment-5391427656
Thanks @SEZ9, glad we're aligned on both points. Agreed the failing checks on the stale head look unrelated to the CHAR/VARCHAR mapping change, and that @davidzollo's earlier approval is conditional on a green `Build` on the synced head. I'll do a fresh full pass as soon as @dik111 syncs with `dev` and pushes the updated head — will flag it if the merge shifts any line numbers around `convert()`. -- 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]
