DanielLeens commented on PR #12209: URL: https://github.com/apache/seatunnel/pull/12209#issuecomment-6557312615
Thanks again. The CI state has advanced since the source-contract discussion: Build is now SUCCESS on this unchanged head. That clears the earlier execution/pending-CI observation only. My latest source objection remains open: the job-env tier passed by JobMaster.resolveLineageConfig can override administrator-controlled OpenLineage settings, including the security/transport controls discussed in my preceding reply. The green Build does not prove those controls safe or clear that blocker. I continue to agree with the related reviewer concern; this is not an approval and does not add another numbered source finding. Please address the already-recorded ownership/allowlist boundary and its regression coverage before final maintainer approval. Checked 2026-11-01T11:27:12.930222+00:00; unchanged head `c8cdfaae7384d50868be61dc3bccf2bb43f3cf5f`. Current checks: [Build](https://github.com/apache/seatunnel/runs/123975562759)=SUCCESS. Base `dev`, compare `diverged`, ahead 27 / behind 51; merge `UNKNOWN` / `REVIEW_REQUIRED`. Drift is recorded independently; it does not itself request synchronization. Daniel has READ permission. This is a CI/merge-gate clarification, not another full source review, effective approval or 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]
