alamb commented on PR #25203: URL: https://github.com/apache/datafusion/pull/25203#issuecomment-5715727031
> I don't think that it should not run the tests in the pr itself, especially for ones that touch that area and add tests > > this means that you have to run locally the entire extended tests, or get your pr approved and then fail on extended tests when trying to merge, and do another round of review I agree this is the tradeoff (if your PR breaks extended tests, your PR takes longer) However, I don't think we can afford to keep running extended tests on each push to the PR Maybe we could also support manually triggering the extended tests, so that people who wanted to run them on CI before the merge queue could do so 🤔 -- 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]
