comphead commented on code in PR #14320: URL: https://github.com/apache/datafusion/pull/14320#discussion_r1947333625
########## docs/source/contributor-guide/testing.md: ########## @@ -58,6 +58,19 @@ Like similar systems such as [DuckDB](https://duckdb.org/dev/testing), DataFusio DataFusion has integrated [sqlite's test suite](https://sqlite.org/sqllogictest/doc/trunk/about.wiki) as a supplemental test suite that is run whenever a PR is merged into DataFusion. To run it manually please refer to the [README](https://github.com/apache/datafusion/blob/main/datafusion/sqllogictest/README.md#running-tests-sqlite) file for instructions. +## 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 relevant +portions of the entire [sqlite test suite]. + +[extended.yml]: https://github.com/apache/datafusion/blob/main/.github/workflows/extended.yml +[sqlite test suite]: https://www.sqlite.org/sqllogictest/dir?ci=tip + Review Comment: ```suggestion ``` -- 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
