alamb commented on PR #15287: URL: https://github.com/apache/datafusion/pull/15287#issuecomment-2746384487
> This PR seems to have broken `main`. It is an easy fix (error messages do not match). > > @alamb -- Extended tests break very frequently these days. We should prioritize completing the work on running them before merge. @ozankabak I think there is a PR waiting on review that would let us more easily run the tests before merge. I am not sure how we would have known to run them in this case though - https://github.com/apache/datafusion/pull/15101 I feel like what would be idea is some workflow that actually ran the tests before merge but that didn't need us to babysit the PR -- something like letting a bot merge it or a merge queue -- 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
