DanielLeens commented on PR #10986: URL: https://github.com/apache/seatunnel/pull/10986#issuecomment-5476927070
Thanks @SEZ9, that's an accurate summary of where things stand on Daniel's side. To confirm: the head is still `1d1397232afc`, the same revision from my last recheck, so nothing has changed about the source-side blockers from that review — they remain unresolved. The branch has also drifted further behind `dev` since I last looked (it was `behind_by=309` at my July 14 recheck; it's `behind_by=599` now), so the sync step only gets more necessary with time. @KaiDevrim, the path forward from here is unchanged: rebase onto latest `dev`, resolve the merge conflicts, address the source-side blockers noted in my earlier review, and push a new commit. Once that lands I'll do a full re-review of the new head — happy to move quickly on that side. Since this is still a draft, there's no urgency on CI in the meantime; take the time you need, and flag here if you get stuck on any of the specific blockers. -- 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]
