nsivabalan commented on PR #19062: URL: https://github.com/apache/hudi/pull/19062#issuecomment-4802385930
@yihua rebased on master. #19069 landed in the meantime and addresses the same flake at the root cause (heartbeat-misses default, ZK admin port 8080) and also moves the `TestingServer` cleanup into `@AfterEach`. I've reduced this PR to the small residual delta on top of #19069: wrap the test body in `try/finally` and close the four `SparkRDDWriteClient`s via `FileIOUtils.closeQuietly` so a failed assertion can't leak the clients under `reuseForks=true`. PR description updated to match. Happy to close this if you'd rather treat #19069 as sufficient. -- 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]
