wombatu-kun commented on PR #16438: URL: https://github.com/apache/iceberg/pull/16438#issuecomment-4505828855
@manuzhang CI on the scoped-down version just failed on a different parameterization of the same assertion — `TestIntegrationDynamicTable.testIcebergSink [1] null` (the unpartitioned/no-branch case, runs first in the class), `IntegrationTestBase.java:255`, only on the Java 21 job. The original commit message documented failures concentrated on `[2] test_branch`, so this looks like a second cause and not the cross-test control-topic contamination the fix addresses. The CI artifact only contained the unit-test log — no per-test integration report, no Connect/Kafka container output — so I can't root-cause from this run. Restored both observability commits onto this branch (`ed0d5c3e`, `8ab75e24`) so the next CI run produces actionable diagnostics. Happy to re-split once we've understood the new failure. (I'll leave #16504 open in parallel for now.) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
