DanielLeens commented on PR #11843: URL: https://github.com/apache/seatunnel/pull/11843#issuecomment-5539353843
Checked the CI run that completed since my last comment: it failed, but on two jobs that have nothing to do with this PR's DM NVARCHAR2 diff — `updated-modules-integration-test-part-1 (11, ubuntu-latest)` failed on `PostgresCDCIT` (a single `Errors: 1` on a Postgres CDC test; this is a known pre-existing intermittent NPE in `DebeziumJsonConverter.serializeValue`, unrelated to the DM connector), and `updated-modules-integration-test-part-7 (8, ubuntu-latest)` failed on `JdbcHanaIT` (a container-startup timeout, also a known unrelated flake). This PR doesn't touch Postgres CDC or the Hana connector at all. Given that and the maintainer sign-off already on record, my conclusion is unchanged: the only thing standing between this and merge is getting a clean CI run, and neither of these failures reflects a problem with the PR itself. A rerun of those two jobs should be enough. -- 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]
