88fantasy commented on PR #12209: URL: https://github.com/apache/seatunnel/pull/12209#issuecomment-6258985989
Thanks @DanielLeens for the review. Addressed on tip a69148fd0: 1. Rebased/merged onto the latest #12208 (openlineage-core @ 4bccb0e55) so the docs Markdown / license-before-H1 failure from the older stack should be gone. 2. Lineage HTTP emission now uses a dedicated bounded pool on CoordinatorService (2 fixed threads, ArrayBlockingQueue of 64, thread name seatunnel-lineage-emission-*, reject = log-and-drop), shared by JobMasters instead of the shared coordinator executor. 3. Per-job lineageEmissions is bounded (64); when full we drop the newest event and warn. Added UT coverage for the drop path. Please take another look when you have a chance. 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]
