DanielLeens commented on PR #11077: URL: https://github.com/apache/seatunnel/pull/11077#issuecomment-6338374312
Thanks @SEZ9 and @hesam-oxe. Agreed: please link the fork Build for the intended head and list which of F1-F8 that head addresses, so both reviewers can assess one concrete version. The current Build check is still a missing-fork-run detection gate, not a maintainer-approval gate. Please start Build from the fork Actions tab and resolve any activation restriction shown there; a source patch or an empty commit is not needed merely to start an available workflow. The collision-identity, pattern-precedence and multi-table coverage items from my previous review remain open until committed fixes can be examined. This is a thread reply, not confirmation that those fixes exist or a new full review. Status evidence: head `8b383c04eeed65c41a14ba756100a33f430124eb`, base `dev`, compare `diverged`, ahead 38, behind 83; GitHub mergeable state `blocked`. -- 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]
