kfaraz commented on PR #18766: URL: https://github.com/apache/druid/pull/18766#issuecomment-3722668662
> @kfaraz I'm wondering if you have a better idea to deal with the timing issue, beyond adding a sleep here. That's the best thing I can think of right now. Thanks for the clarification, @gianm ! The only alternative I can think of would be to emit a metric in `S3TaskLogs.pushTaskLog()` but if you feel that seems like an overkill, we can proceed with a sleep to unblock this PR. I think there are certain APIs for which there is no option but to have a test-sleep-repeat pattern. We should consider putting something similar to `ITRetryUtil.retryUntil()` in `EmbeddedClusterApis`. -- 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]
