schizophrenicmaniac commented on PR #40740: URL: https://github.com/apache/beam/pull/40740#issuecomment-6450261965
@aIbrahiim You're right, the job was actually broken by the `Unsupported class file major version 69` error. Master now has the same java-version pin (plus the settings.xml step), and I merged master and resolved the conflict in the workflow file, so the pin is no longer part of this PR's diff. The test changes weren't the cause of the failures in those logs. They're for separate flakiness risks I noticed in the IT setup. `findAvailablePort()` reserves host ports and releases them before the container binds them, so another process can take them in between. The read tests sleep a fixed 15s before checking the counter and never cancel the unbounded read pipeline. And the SEMP queue/subscription setup ignores errors, so if SEMP isn't ready yet the test only fails later with a wrong count. -- 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]
