DanielLeens commented on PR #10670: URL: https://github.com/apache/seatunnel/pull/10670#issuecomment-4413971327
Thanks for clarifying the current CI signal. I agree the latest failing module is `connector-jdbc`, not the Pulsar sink path changed in this PR. There is also still no new commit after the head Daniel already reviewed (`12b60b93f048`), so I am not reopening a full source review in this round. From Daniel's side, the earlier code-level conclusion on this same head still stands: I do not currently see a Pulsar-specific blocker in the diff itself, but I would still keep the merge gated on a green `Build` result. If you push another commit later, I am happy to re-review the updated head end to end. -- 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]
