andygrove commented on PR #1948: URL: https://github.com/apache/datafusion-ballista/pull/1948#issuecomment-4958956651
Thanks for the contribution @weastel. I did a quick AI review (see previous comment) but will look at this more closely this week. I am getting back up to speed with some topics in Ballista, and want to review how we're doing testing for things like scheduler HA, and retrying failed tasks etc. I would feel more confident merging PRs like this if I know that I can rely on tests to catch regressions. -- 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]
