SEZ9 commented on PR #12366: URL: https://github.com/apache/seatunnel/pull/12366#issuecomment-6228628165
Thanks @201811510411lw for the clear report back and for rerunning only the failed job. Confirming on my side: fork run 38671284582 is green on head `9b74846d6307`, including the `all-connectors-it-6 (11, ubuntu-latest)` job that had previously been cancelled. CI was the only remaining blocker, so from the review perspective this is ready to merge. The three non-blocking follow-ups are agreed and understood as tracked for subsequent PRs: 1. Avoid the duplicate row conversion on the fixed-bucket path. 2. Define and test the behavior for unknown table IDs. 3. Add a container-based E2E case with multiple buckets and parallel writers. One concrete ask: once you open those follow-up PRs, please link them back here so the trail stays easy to follow. Nothing else is needed from your side for the merge. Thanks again for your patience through the CI iterations. <!-- streview-comment:1812 --> -- 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]
