Omega359 commented on code in PR #14320: URL: https://github.com/apache/datafusion/pull/14320#discussion_r1930912117
########## docs/source/contributor-guide/testing.md: ########## @@ -56,6 +56,17 @@ DataFusion's SQL implementation is tested using [sqllogictest](https://github.co Like similar systems such as [DuckDB](https://duckdb.org/dev/testing), DataFusion has chosen to trade off a slightly higher barrier to contribution for longer term maintainability. +## Extended Tests + +In addition to the standard CI test suite that is run on all PRs prior to merge, +DataFusion has "extended" tests (defined in [extended.yml]) that are run on each +commit to `main`. These tests rarely fail but take significantly longer to run +than the standard test suite and add important test coverage such as that the +code works when there are hash collisions as well as running the entire [sqlite +test suite](https://www.sqlite.org/sqllogictest/dir?ci=tip). Review Comment: Almost the entire suite ... I do remove the evidence and index/delete/ folders. -- 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
