DanielLeens commented on PR #10400: URL: https://github.com/apache/seatunnel/pull/10400#issuecomment-6538171328
GitHub now reports an actual content conflict (CONFLICTING / DIRTY), superseding the earlier no-conflict snapshot on this unchanged head. Please reconcile the latest base while preserving both this PR's change and the intervening upstream work, then obtain required checks on the resulting commit. This request is based on the conflict, not the number of commits behind. Existing-head CI cannot validate that future resolution. My prior source conclusion remains clear for the reviewed scope; optional recommendations remain optional. The new conflict and current CI/repository approval gates are separate from that technical conclusion. The earlier wording “updating my review to Approve” was inaccurate: the published event was COMMENT, not an effective APPROVED event. Sorry for that ambiguity. Checked 2026-10-31T03:04:29.988724+00:00; unchanged head `2881111d572d198ad5f113f9f69f3fdc7735bb43`. [Signal pull request update](https://github.com/apache/seatunnel/actions/runs/40299356987/job/121493142369)=`SUCCESS`; [Build](https://github.com/apache/seatunnel/runs/121504227374)=`SUCCESS`. This is a discussion/CI clarification, not another full source review or an effective maintainer approval. Daniel has READ repository permission; a write-capable maintainer owns required approval/dismissal. -- 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]
