88fantasy commented on PR #12210: URL: https://github.com/apache/seatunnel/pull/12210#issuecomment-6258986169
Thanks @DanielLeens for the review. Addressed on tip 938ac20c7: 1. Merged onto the latest #12208 (openlineage-core @ 4bccb0e55); docs Markdown issues inherited from the older stack should be resolved. 2. Added ASF license headers to both test META-INF/services LineageBackend resources. 3. Renamed ShadedArtifactIT → TestShadedArtifactIT (and updated references) so it satisfies both the Test* UT naming rule and failsafe *IT. 4. registerJobStatusHook is now try/caught in FlinkLineageHooks.register; on failure we emitAbort for the already-sent START and return false. Please re-review when convenient. CI has been re-triggered on the fork. -- 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]
