jayzhan211 commented on PR #14689: URL: https://github.com/apache/datafusion/pull/14689#issuecomment-2676170935
Making extended tests optional seems like a better approach. This way, for minor changes or cases where we're confident in the outcome, we can choose to skip the tests. https://github.com/apache/datafusion/issues/14319 We also need to add more tests to SQLLogicTest to improve coverage. This failure highlights the need for additional tests to address the gap. https://github.com/apache/datafusion/pull/14824/files I think we can do both -- 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
