ozankabak commented on PR #14689: URL: https://github.com/apache/datafusion/pull/14689#issuecomment-2676043465
CI broke after this PR. We have this extended-tests-failing-after-merge happening very frequently recently. Maybe the decision to defer extended tests to post-merge was a wrong one. Committers are now unaware that their PRs can break main, and they only get to know this after the fact. We should discuss how to mitigate this. If we don't see an obvious solution, it may be prudent to go back to the inefficient but safe run-everything-in-the-PR mode. @alamb and @jayzhan211, what do you think? -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
