SEPURI-SAI-KRISHNA commented on PR #12416: URL: https://github.com/apache/seatunnel/pull/12416#issuecomment-5758456760
Noting the red badge so it does not cost a reviewer time: the failure is not reachable from this diff. One leg failed, `unit-test (11, windows-latest)`: ``` PayPalClientTest.closeWakesRetryWait:416 expected: <true> but was: <false> ``` That line is `assertTrue(arrived.await(NETWORK_WAIT_SECONDS, TimeUnit.SECONDS))`, a 15 second wait on a loopback HTTP request, and it timed out at 15.029s on a Windows runner. This diff is two unit test files under `connector-mqtt` and shares nothing with `connector-http-paypal`. The same class has already been widened once for this reason in #12370, and #12381 is open and widens this exact file again. What did pass on the same run: the other three `unit-test` legs, and all eight `updated-modules-integration-test-part-N` shards on both JDK 8 and JDK 11. The tests this PR actually changes ran on the failing leg too and were green there: ``` Tests run: 11 ... MqttSourceConfigTest Tests run: 12 ... MqttSourceTest ``` Happy to rerun that leg if a reviewer would rather see the badge green, but it will stay a coin flip until #12381 or something like it lands. -- 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]
