Ma77Ball opened a new pull request, #5853: URL: https://github.com/apache/texera/pull/5853
### What changes were proposed in this PR? - Add `awaitDependency` to `FileService`, an exponential-backoff retry (6 attempts from 200ms, ~6s total) with an injectable sleep, mirroring `LakeFSStorageClient.retryWithBackoff`. - Wrap the two `S3StorageClient.createBucketIfNotExist` calls in `FileService.run` with it, so a slow-to-start MinIO/S3 no longer aborts file-service startup. - Leave `LakeFSStorageClient.healthCheck()` on its existing inner retry (unchanged). - Add `FileServiceSpec` covering immediate success, retry-then-success, give-up-with-cause, and interrupt-fails-fast. ### Any related issues, documentation, discussions? Closes: #5852 ### How was this PR tested? - Run `sbt "FileService/testOnly org.apache.texera.service.FileServiceSpec"` and expect 4 passing tests: immediate success runs the operation once and never sleeps; retry-then-success records delays `List(200, 400)` before succeeding on the 3rd try; give-up rethrows after `maxAttempts` with the original exception as `getCause`; interrupt restores the interrupt flag and fails fast. - This environment hits a pre-existing JaCoCo instrumentation error on an unrelated class (`DatasetFileNodeSerializer`) under the instrumented test classpath, so the spec was verified locally against the un-instrumented classpath (4/4 pass) and relies on CI's JDK/JaCoCo combo for the standard run. ### Was this PR authored or co-authored using generative AI tooling? Co-authored with Claude Opus 4.8 in compliance with ASF -- 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]
